Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 566440

Summary: Handle specific "connection lost" problem that can appear when using Amalgam with collaborative environment (Team for Capella for example)
Product: [Modeling] Amalgam Reporter: Laurent Redor <laurent.redor>
Component: UIAssignee: Project Inbox <amalgam.ui-inbox>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: philippe.dul
Version: unspecified   
Target Milestone: 1.12.0   
Hardware: PC   
OS: Windows 10   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=566437
https://git.eclipse.org/r/c/amalgam/org.eclipse.amalgam/+/168296
https://git.eclipse.org/c/amalgam/org.eclipse.amalgam.git/commit/?id=849bca3217db10bc7a38422e85db20d56848a784
Whiteboard:
Bug Depends on: 566274    
Bug Blocks:    

Description Laurent Redor CLA 2020-08-27 05:18:56 EDT
As for Sirius bug 566274 and Capella bug 566437, in a collaborative environment, when the data is host on a server, the connection with the server can be lost. Amalgam is not ready to handle this context. If Amalgam tries to access to data, for example with an async refresh of the UI that needs information from data to be refreshed, a org.eclipse.net4j.util.lifecycle.LifecycleException is thrown with the message "Not active: CDOTransactionImpl". This kind of errors is showed to the end-user instead of silently catch it. Indeed, in this situation, the UI should be refreshed with "blank data" instead of displaying an "incomprehensible" popup to the end-user.

In bug 566274, a specific query has been added to "detect" this specific case. The goal of this issue is to use this specific query to avoid this kind of problem.
Comment 1 Laurent Redor CLA 2020-08-27 05:24:23 EDT
Proposition of fix: https://git.eclipse.org/r/c/amalgam/org.eclipse.amalgam/+/168296 [566440] Silently catch the exception corresponding to a connection lost
Comment 2 Eclipse Genie CLA 2020-08-27 05:24:39 EDT
New Gerrit change created: https://git.eclipse.org/r/c/amalgam/org.eclipse.amalgam/+/168296