Community
Participate
Working Groups
SemanticChangedCommand is called in precommit to do local refresh of DRefreshable even in auto-refresh and do refresh of EditPart (in precommit also). The local/partial commit in SemanticChangedCommand should be called only in manual-refresh in a separate precommit. And Editpart refresh should be handled through DiagramEventBroker.
Can you provide more details about the consequences (with a scenario)? You describe a solution but this issue will be complicated to test during the validation without more details.
Esteban, from your description this looks like a performance issue (causing too many unneeded DRefreshable.refresh() and EditPart.refresh() calls). Is this correct or can this also produce invalid results (and not just slowdowns)? If this is about performance, please add the corresponding keyword, and if possible some measures (ideally times, otherwise suspiciously high invocation counts for some methods). Note that this may be one of the causes for what is described in bug #443518.
This bugzilla is to show that some refresh is called in bad places. Fixing that will improve performance, I will try to do some performance measures.
