Community
Participate
Working Groups
This is a generic task for performance-related improvements. Most performance issues on Big models (> 10k elements) are related to CrossReferencers (Stereotype application, object usages, ...) and too early calculations in commands (For example, when handlers test whether they should be enabled, they should avoid building the full command which would be executed. Otherwise, opening a context menu can take a lot of time) - Usage to CrossReferencer should be limited to strict necessity - Generic command-based handlers should be avoided (Because they build the command then call canExecute(), which can be really expensive) - Facets and Customizations should avoid expensive queries to determine whether they are applied or not, because EMF Facet sends hundreds of refresh requests (ModelExplorer). Especially, queries such as "isDiagramContainer" should not browse the full resource set to retrieve all existing Diagrams. For handlers, it is often possible to make a really simple test in the "isEnabled" method, and return true when in doubt (It is better to have a handler which is always enabled and sometimes does nothing, than a really accurate handler which will take tens of seconds to compute isEnabled() and will almost never be actually called)
- Modified Hide and Delete handlers to improve performances (The "isEnabled" has been simplified to avoid freezing the popup menu on big models) - Changed the implementation of PropertyHelperAdvice (In Composite and SysML IBD diagrams) to call the cross referencer as late as possible (Only when necessary). Previously, it was always called and most of the time not used. c4fb8e980348c472d4f849187a1526d902c9ca89 5edafe2b18a98c06fbc9aad3a276e5512a4f0239 ed760c6f9e8e89da5e2fa57722ce8367ec4fb349 Small regression to be fixed: the Delete handler is enabled too often (Especially, it is enabled on root semantic elements, although it does nothing) Commit 767fe2bde06aad86789a9a5419665b59854e3580 fixes some issues on the EMF Facet queries (But the commit is associated to Bug 363835)
> Small regression to be fixed: the Delete handler is enabled too often (Especially, it is enabled on root semantic elements, although it does nothing) Fixed in ce8aa9e818698b36aef8567c7a581b6e82d6cfbe Pushed to master
