Summary: | [api] RSE needs improved refresh policies | ||
---|---|---|---|
Product: | [Tools] Target Management | Reporter: | Martin Oberhuber <mober.at+eclipse> |
Component: | RSE | Assignee: | dsdp.tm.rse-inbox <tm.rse-inbox> |
Status: | NEW --- | QA Contact: | Martin Oberhuber <mober.at+eclipse> |
Severity: | enhancement | ||
Priority: | P3 | CC: | ddykstal.eclipse, dmcknigh, eclipse, kmunir, tobias.schwarz, uwe.st, wbprio |
Version: | 1.0.1 | Keywords: | api, investigate |
Target Milestone: | Future | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: | |||
Bug Depends on: | 186994, 186991, 186997 | ||
Bug Blocks: | 150954, 160768, 170922, 181145 |
Description
Martin Oberhuber
2007-02-06 08:12:58 EST
This is a must-have for our product. Assigning M6 since it involves API change. Shooting for right after M6 It turns out that most problems of refresh (losing selection and expand state) are due to the SystemDeferredTreeContentManager, which schedules jobs for getting remote data from slow systems. Bug #150954 comment 13 has a suggestion how this could perhaps be fixed, but the solution does not seem elegant. For the longer term future, we should rather look at what the Platform Debug and DSDP-DD teams have been working on in order to support custom refresh policies. Until this is done, the recommended workaround for clients is to do their own asynchronous updates, and indicate this with supportsDeferredQueries==false in their SubsystemConfiguration as well as their ISystemViewElementAdapter. They can further optimize Refreshes by not implementing IRemoteElementAdapter in their ISystemViewElementAdapter. Bug #150954 comment 12 has a description of an algorithm that clients can use. An important point is the separation of "Refresh Viewer Only" vs. "Refresh getting new data" that can be accomplished by a local subclass of SystemResourceChangedEvent. Important bug fixes for making this work have been done in bug #186991, and bug #181145. Since these were the most problematic road blockers, I'm moving this to P3 / Future for now. |