Summary: | [CommonNavigator] Caching should be removed from CommonViewer; address over-calling of getSelection() | ||
---|---|---|---|
Product: | [Eclipse Project] Platform | Reporter: | Francis Upton IV <francisu> |
Component: | UI | Assignee: | Francis Upton IV <francisu> |
Status: | NEW --- | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | bokowski, eclipse, remy.suen, ylulu |
Version: | 3.4 | Keywords: | performance |
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Linux | ||
Whiteboard: |
Description
Francis Upton IV
2008-02-12 04:16:52 EST
Yasser, I have opened this bug to try to solve the performance problem which motivated the addition of caching to the CommonViewer. For a bit of context, the addition of caching to the CommonViewer (bug 144294) had a problem (bug 197113). My feeling is that caching is a bad idea, because it's hard to get right on all platforms can could potentially break in the future with SWT changes, etc. Therefore, I'm proposing with this bug that we remove unnecessary calls to getSelection() in the CommonNavigator. I think doing this will solve your performance problem. Can you provide some additional data as to where the peformance problems you had were, if they were not in the stack trace that I provided? Thanks, Francis Boris, You will see the message to Yassir above. This is in reference to the hallway conversation we had at EclipseCon. We need to understand exactly how Yassir is using the CN (what method(s) he's calling that are causing the problem) so that we can do a solution in the CN to improve performance so that it's acceptable to him. Can you get this information? Once I have this, then I can work on a patch to the CN that reduces the number of calls to getSelection() and then we can remove the caching and then fix the critical bug 197113 (which must be dealt with for 3.4). The big items that call getSelection() are in filling the action bars and setting up the context menu. For the former, a selection provider can be given that uses a selection is currently available before the filling action bars starts. For the latter, the code can be changed to minimize the calls. This bug hasn't had any activity in quite some time. Maybe the problem got resolved, was a duplicate of something else, or became less pressing for some reason - or maybe it's still relevant but just hasn't been looked at yet. If you have further information on the current state of the bug, please add it. The information can be, for example, that the problem still occurs, that you still want the feature, that more information is needed, or that the bug is (for whatever reason) no longer relevant. |