Bug 296482 - Investigate splitting up the incremental builder
Summary: Investigate splitting up the incremental builder
Status: CLOSED WONTFIX
Alias: None
Product: PDE
Classification: Eclipse Project
Component: API Tools (show other bugs)
Version: 3.6   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: PDE API Tools Inbox CLA
QA Contact:
URL:
Whiteboard: stalebug
Keywords: performance
Depends on:
Blocks:
 
Reported: 2009-11-30 10:29 EST by Michael Rennie CLA
Modified: 2019-09-09 02:35 EDT (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2009-11-30 10:29:38 EST
In its current state our incremental builder performs all of its actions in one shot when the builder is called (as expected). We could however look at breaking the builder up into separate parts, say the compatibility, usage, leak and tag analyzers. The benefit of this would be that we could run each part separately from the others, since each part has different requirements wrt the types it has to scan.

For example:

1. the compatibility analyzer could be run as a Java builder participant, since it only cares about structurally changed types

2. the tag analyzer could also be run as a participant since it only cares about java source that has changed (structurally and non-structurally)

3. The leak and usage parts could be run the way they are now (from the API builder), but we could leverage (record somehow) the changed source file names when the participant runs (from 1 and 2 above), that way we do not have to compute them ourselves, we would only have to look up dependent types whose API description has changed. This would allow us to do less work to know if a leak or usage scan should be a no-op and save us cycles re-doing the same work that the Java builder has already done.
Comment 1 Markus Keller CLA 2010-01-29 10:46:08 EST
That sounds like a very good idea.
Comment 2 Eclipse Webmaster CLA 2019-09-06 16:18:14 EDT
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.
Comment 3 Julian Honnen CLA 2019-09-09 02:35:26 EDT
Please remove the stalebug flag, if this issue is still relevant and can be reproduced on the latest release.