-
Notifications
You must be signed in to change notification settings - Fork 100
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
DiagramServices::getEditPart(DDiagramElement) can cause deadlock #2894
Comments
mPorhel
added a commit
to mPorhel/capella
that referenced
this issue
Sep 9, 2024
Change-Id: I653a0e579d58abf71f44ceb8eb85179cc07d5eb5 Signed-off-by: Maxime Porhel <[email protected]>
With the provided PR, the issue is no more reproduced with the scenario we have with Team for Capella. |
mPorhel
added a commit
to mPorhel/capella
that referenced
this issue
Sep 25, 2024
Change-Id: I653a0e579d58abf71f44ceb8eb85179cc07d5eb5 Signed-off-by: Maxime Porhel <[email protected]>
GlennPlou
added a commit
to GlennPlou/capella
that referenced
this issue
Oct 15, 2024
Signed-off-by: Maxime Porhel <[email protected]> Signed-off-by: Glenn Plouhinec <[email protected]>
lredor
pushed a commit
that referenced
this issue
Oct 24, 2024
Signed-off-by: Maxime Porhel <[email protected]> Signed-off-by: Glenn Plouhinec <[email protected]>
GlennPlou
added a commit
to GlennPlou/capella
that referenced
this issue
Nov 20, 2024
Signed-off-by: Maxime Porhel <[email protected]> Signed-off-by: Glenn Plouhinec <[email protected]>
tguiu
pushed a commit
that referenced
this issue
Nov 20, 2024
Signed-off-by: Maxime Porhel <[email protected]> Signed-off-by: Glenn Plouhinec <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In some situations, deadlock can occur during refresh of diagrams having the capability to display functional chains or physical paths: ie diagrams with Sirius post refresh extensions :
The detected performance issue or potential deadlock comes from the DiagramServices::getEditPart(DDiagramElement) which uses an helper using Display.syncExec method to get the ActiveEditor.
This also means that a refresh occuring on a non active editor will not refresh the edit parts.
Reproduction scenario exists with Team for Capella:
The text was updated successfully, but these errors were encountered: