Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 360522 - [Property View - Customization] The copy of an existing customization can take a lot of time
Summary: [Property View - Customization] The copy of an existing customization can tak...
Status: RESOLVED FIXED
Alias: None
Product: Papyrus
Classification: Modeling
Component: Core (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Camille Letavernier CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-11 05:42 EDT by Camille Letavernier CLA
Modified: 2011-10-11 07:54 EDT (History)
0 users

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Camille Letavernier CLA 2011-10-11 05:42:51 EDT
To edit a property configuration contributed by a plug-in, we first need to make a copy of it (As the plug-in contributions are read-only).

However, the current implementation of the copy action is really expensive (It takes a few minutes for UML). The problem comes from the fact it is not possible to list a folder's contents when this folder is stored in an archive (Such as a Jar plug-in). Thus, we need to rely on the EcoreUtil#resolveAll method to retrieve all the resources related to the configuration.

Although we're only interested on the file names, this methods fully loads all the resources in memory, which takes a lot of times when it comes to *.xwt files (~500 files for UML, with an expensive parsing operation). Moreover, we know that these *.xwt files do not contain any useful cross-reference to other resources : it is not necessary to load them.

We should re-implement a method similar to EcoreUtil#resolveAll, while ignoring the *.xwt files.
Comment 1 Camille Letavernier CLA 2011-10-11 05:53:01 EDT
Fixed in r5779 (Branch 0.8.X)
Merged to the trunk in r5780
Comment 2 Camille Letavernier CLA 2011-10-11 07:54:10 EDT
> Fixed in r5779 (Branch 0.8.X)
> Merged to the trunk in r5780

Resolved.