Bug 506408 - Allow properties views to be refreshed on purely graphical model changes without compromising performance
Summary: Allow properties views to be refreshed on purely graphical model changes with...
Status: NEW
Alias: None
Product: Sirius
Classification: Modeling
Component: Properties (show other bugs)
Version: 4.1.0   Edit
Hardware: All All
: P3 major (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords: performance, triaged
Depends on:
Blocks:
 
Reported: 2016-10-24 04:02 EDT by Pierre-Charles David CLA
Modified: 2017-08-16 09:28 EDT (History)
1 user (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Pierre-Charles David CLA 2016-10-24 04:02:52 EDT 
Before https://bugs.eclipse.org/bugs/show_bug.cgi?id=506268 we used to refresh the contents of properties view on every change in any model inside the Sirius session. This had bad performance implications in the case where the only changes where graphical (in the sense that they touch only the Sirius representation models).

The emergency fix done for #506268 was to completely disable properties view refreshes if the only changes in a transaction concerned the representation models (with a hidden system flag to revert to the previous behavior if really needed). This means by default we reduced the functionality, in the few legitimate cases where the property views content depend on the state of the representation.

This ticket is about finding a better way to avoid the performance issue fixed by #506268 while still retaining the previous expressiveness.
Comment 1 Pierre-Charles David CLA 2016-10-27 09:16:01 EDT 
See also https://bugs.eclipse.org/bugs/show_bug.cgi?id=506268#c5 which describes a problematic behavior which may be caused by the "emergency fix" made for #506268.