Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 465576

Summary: Performance enhancements on sirius.query.legacy
Product: [Modeling] Sirius Reporter: Philippe Dul <philippe.dul>
Component: CoreAssignee: Laurent Redor <laurent.redor>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: laurent.redor, maxime.porhel, pierre-charles.david
Version: 2.0.4Keywords: triaged
Target Milestone: 3.0.0M7   
Hardware: PC   
OS: Windows 7   
See Also: https://git.eclipse.org/r/45121
https://git.eclipse.org/c/sirius/org.eclipse.sirius.legacy.git/commit/?id=1f89ace4b364674101c98e32e0a2bf0716e76703
Whiteboard:

Description Philippe Dul CLA 2015-04-27 08:47:42 EDT 
There is a performance problem on query.legacy.

Query.legacy allows to use *.mt files in addition of java services for Sirius IJavaExtension definition. For instance, when a IJavaExtension org.sample.ExampleServices is defined in the odesign, query.legacy will try to find a MT template named org.sample.ExampleServices.mt in all plugins required by the plugin, then will load the java class using a Class.forName(org.sample.ExampleServices).

With a complex hierarchical plugin dependencies like Capella, the lookup of MT files can take up to 8 seconds whereas we don't use any MT files. It could be interresting to inverse the lookup mechanism to allow better performances in modeller which doesn't use MT files: if java service is found, there is no needs to lookup for MT files.
Comment 2 Laurent Redor CLA 2015-04-27 11:40:43 EDT 
The fix proposed by Philippe is available in the new 1.1.0 version of oes.query.legacy.

This new version is available here [1].
To avoid unexpected update the old URL [2] always points to the version 1.0.0, that is also available here [3].

[1] http://download.eclipse.org/sirius/updates/legacy/1.1.0
[2] http://download.eclipse.org/sirius/updates/legacy
[3] http://download.eclipse.org/sirius/updates/legacy/1.0.0
Comment 3 Pierre-Charles David CLA 2015-05-21 05:36:46 EDT 
Improvements confirmed by reporter.
Comment 4 Pierre-Charles David CLA 2015-06-24 11:15:31 EDT 
Available in Sirius 3.0.0. See https://wiki.eclipse.org/Sirius/3.0.0.