Community
Participate
Working Groups
See bug 60117
Created attachment 17620 [details] Start of a fix Not finished yet. Attaching this patch here so it doesn't get lost.
This will also fix bug 58836
See bug 65490
This patch should be part of the M7 performance work. It will affect performance in the following ways: 1. Reduce the number of jobs required to populate the marker views. The current code requires 2 jobs per view. In its current form, the patch uses 1 job per view plus 1 global search job. 2. Reduce memory overhead. Since all marker views would share the same model, the cached form of any marker that appears in more than one marker view would only be stored once. This would have a minor effect on a single workbench window (since markers rarely show up in, say, both the problems and tasks view), but would have big improvements for multiple workbench windows (especially with respect to the number of allocated coallator keys). 3. Greatly improve the perceived performance of the marker views. Since the view can respond to marker deltas, it can display markers as soon as they're available rather than waiting until the build completes. It is hard to measure this performance improvement exactly (since it depends entirely on the duration of the build and when the builder fires off deltas), but it is likely to improved perceived performance by an order of magnitude.
Just to be thorough, we need to be very clear in describing what we are fixing and why in performance areas like this and will require real tests that prove that: A) we've gotten faster and/or smaller B) that percieved performance increases do not increase memory usage etc... C) they were worth doing and we can show this
Tod: this is the patch we were talking about. I was wondering if you could help me to finish it.
Created attachment 28205 [details] Zipped version of these classes As this is getting more and more divergent I exported the classes with Stefans changes to this zip
The rework in 3.2 was such that this is no longer relevant.