Bugzilla – Full Text Bug Listing
| Summary: | Launching a JUnit test takes minutes writing platform.xml | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Product: | [Eclipse Project] PDE | Reporter: | Johannes Rieken <johannes_rieken> | ||||||
| Component: | UI | Assignee: | Curtis Windatt <curtis.windatt.public> | ||||||
| Status: | RESOLVED FIXED | QA Contact: | |||||||
| Severity: | major | ||||||||
| Priority: | P4 | CC: | curtis.windatt.public, daniel_megert | ||||||
| Version: | 3.7 | Keywords: | performance, polish | ||||||
| Target Milestone: | 3.7 M7 | ||||||||
| Hardware: | PC | ||||||||
| OS: | Mac OS X - Carbon (unsup.) | ||||||||
| Whiteboard: | |||||||||
| Attachments: |
|
||||||||
|
Description
Johannes Rieken
Please take a look during M6. This bug makes using Eclipse a real pain. We are spending a decent amount of time creating a configuration (platform.xml) for the update.configurator. Most people are launching with the simple configurator (not update). I think in those cases we don't need to generate the file. Improving performance for the actual file generation would be more difficult. We need to initialize all the features in the workspace so we can determine which ones are associated with the launch. The actual writing of the file seems a little slow as well, not sure whether there are improvements we can make there. Created attachment 190005 [details]
Prototype to avoid creating the platform.xml
This code moves where we create the platform.xml to inside the config.ini creation code. This allows us more flexibility in determining what bundles are being launched (and therefore which configurator is being used).
Note that this patch does not create a platform.xml if BOTH simple configurator and update configurator are in the bundle list. I'm not sure if this is an issue as the config.ini settings will be set to use only simple configurator.
>We are spending a decent amount of time creating a configuration (platform.xml)
>for the update.configurator.
Is this work also done if I unchecked 'Clear the configuration area before launching'? Also, would it help to use an existing config.ini as template?
(In reply to comment #4) > >We are spending a decent amount of time creating a configuration (platform.xml) > >for the update.configurator. > Is this work also done if I unchecked 'Clear the configuration area before > launching'? According to Johannes, this doesn't help. (In reply to comment #5) > According to Johannes, this doesn't help. Does the patch improve anything? No changes planned for M6. (In reply to comment #6) > (In reply to comment #5) > > According to Johannes, this doesn't help. > > Does the patch improve anything? I don't think Johannes can use/apply the patch. Please connect with Johannes and provide a patched PDE plug-in, so that he can test it. Is there progress here? The provided patches couldn't be applied and I am worried that this will slip. Created attachment 191758 [details]
Fixed Patch
The previous patch was missing some changes.
I have committed the patch to HEAD. Tonight there will be a nightly build that Johannes can test on. Johannes, please test using: http://download.eclipse.org/eclipse/downloads/drops/N20110323-2000/ Using the nightly build shows an improvement. The very first launch was a bit slow (verifying launch attribute) but subsequent launches (also different launch configs) are fast now. Thanks This was released to the I build. Closing as FIXED. |