Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

A proposal for improving the WorkspaceSynchronizer #10

Open
pcdavid opened this issue Sep 4, 2022 · 1 comment
Open

A proposal for improving the WorkspaceSynchronizer #10

pcdavid opened this issue Sep 4, 2022 · 1 comment
Labels
bugzilla Issues migrated from the old Eclipse Bugzilla

Comments

@pcdavid
Copy link
Contributor

pcdavid commented Sep 4, 2022

  • πŸ†” Bugzilla ID: #285263
  • πŸ“˜ Project: Modeling / EMF Services / Transaction
  • πŸ—“ Created: 2009-07-31T11:29:34Z
  • ❓ Status: NEW /
@pcdavid pcdavid added the bugzilla Issues migrated from the old Eclipse Bugzilla label Sep 4, 2022
@pcdavid
Copy link
Contributor Author

pcdavid commented Sep 4, 2022

Comment #0 on Fri Jul 31 2009 13:29:34 GMT+0200 (Central European Summer Time):

When deleting a resource from workspace, the workspace synchronizer will fire handleResourceChanged events events on all resources though they havent been changed.

This is because in the workspace synchronizer in the processDelta operation in the IResourceDelta.CHANGED case does not check if the contents actually have been changed.

I find this a bit myself esoteric but it seems thats the eay how resource change events work with eclipse: When a resource is deleted the ResourcePlugin puts the entire workspace into the resource delta one is required to visit in order to figure out in what the actual change consists in.

To check if a resource has actualy changed on has the check

if ((delta.getFlags() & IResourceDelta.CONTENT) != 0) [1]

This would avoid a lot of unload and reload operations, especialy when it comes to GMF after a diagram is deleted and this events ripples through all editing domains with their attached resource sets causing the lag



[1] http://www.eclipse.org/articles/Article-Resource-deltas/resource-deltas.html

Comment #1 on Sat May 14 2022 15:51:44 GMT+0200 (Central European Summer Time):

Eclipse EMF Transaction is moving away from this bugs.eclipse.org issue tracker to https://github.com/eclipse/emf-transaction.

If this issue is relevant to you and still present in the latest release:

* Create a new issue at https://github.com/eclipse/emf-transaction/issues/.
  * Use as title in GitHub the title of this Bugzilla ticket (may include the bug number or not, at your own convenience)
  * In the GitHub description, start with a link to this bugzilla ticket
  * Optionally add new content to the description if it can helps towards resolution
* Update bugzilla ticket
  * Add to "See also" property (up right column) the link to the newly created GitHub issue
  * Add a comment "Migrated to <link-to-newly-created-GitHub-issue>"
  * Set status as CLOSED MOVED

All issues that remain open will be automatically closed next week or so. Then the Bugzilla component for EMF Transaction will be archived and made read-only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from the old Eclipse Bugzilla
Projects
None yet
Development

No branches or pull requests

1 participant