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

Bug 329133

Summary: Better Exception message for lock conflicts
Product: [Modeling] EMF Reporter: Eike Stepper <stepper>
Component: cdo.coreAssignee: Martin Fluegge <martin.fluegge>
Status: CLOSED FIXED QA Contact: Eike Stepper <stepper>
Severity: enhancement    
Priority: P3 Flags: stepper: review+
Version: 4.0   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard: Lighter, Faster and Better
Attachments:
Description Flags
Patch v1
none
Patch v2 none

Description Eike Stepper CLA 2010-10-30 07:10:46 EDT 
[ERROR] Conflict with org.eclipse.emf.cdo.internal.common.id.CDOIDRevisionDeltaLockWrapper@643cb077
org.eclipse.net4j.util.concurrent.TimeoutRuntimeException: Conflict with org.eclipse.emf.cdo.internal.common.id.CDOIDRevisionDeltaLockWrapper@643cb077
	at org.eclipse.net4j.util.concurrent.RWLockManager.lock(RWLockManager.java:304)
	at org.eclipse.net4j.util.concurrent.RWLockManager.lock(RWLockManager.java:117)
	at org.eclipse.emf.cdo.internal.server.TransactionCommitContext.lockObjects(TransactionCommitContext.java:746)
	at org.eclipse.emf.cdo.internal.server.TransactionCommitContext.write(TransactionCommitContext.java:388)
Comment 1 Martin Fluegge CLA 2010-11-09 13:04:58 EST 
Created attachment 182739 [details]
Patch v1

Well, because the description was a bit short I am not sure whether you only meant the message or also preferred a change the type of the Exception class. 

Because it is easier to fix I assumed you had the first case in mind ;)
Comment 2 Eike Stepper CLA 2010-11-13 03:17:02 EST 
Created attachment 183072 [details]
Patch v2

No, I really meant to add a nice toString() method to the wrapper class ;-)
Comment 3 Martin Fluegge CLA 2010-11-13 04:42:42 EST 
Patch v2 committed to HEAD.
Comment 4 Eike Stepper CLA 2011-06-23 03:39:05 EDT 
Available in R20110608-1407