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

Bug 315138

Summary: Incorrect Commit Order Calculation
Product: z_Archived Reporter: Gordon Yorke <gordon.yorke>
Component: EclipselinkAssignee: Project Inbox <eclipselink.foundation-inbox>
Status: NEW --- QA Contact:
Severity: normal    
Priority: P2 CC: eclipselink.foundation-inbox, tom.ware
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
See Also: https://bugs.eclipse.org/bugs/show_bug.cgi?id=270762
Whiteboard:
Attachments:
Description Flags
Potential Patch
none
Recreation of problem none

Description Gordon Yorke CLA 2010-05-31 16:22:46 EDT 
A node in the commit order calculation is only visited once.  Should a descriptor be referenced by multiple Classes it may not have it's commit order updated in relation to the second reference resulting in incorrect commit order for some of the objects resulting in failed deletes.
Comment 1 Gordon Yorke CLA 2010-06-09 14:54:40 EDT 
Created attachment 171570 [details]
Potential Patch

The attachment contains proposed updates to improve the commit order calculation
Comment 2 Tom Ware CLA 2010-11-18 10:47:28 EST 
I have been unable to recreate this issue in a way that does not also depend on a solution for bug 330580.  Please post a recreation if you can think of one.  I will attach the object model I have that comes the closest.
Comment 3 Tom Ware CLA 2010-11-18 11:15:30 EST 
Created attachment 183395 [details]
Recreation of problem

Attaching a zip file that recreates the problem.  To fix the issue, for this recreation in a general way, the fixes for bug 330580 and bug 315141 will be reqquired.

To run the recreation, import the project into an Eclipse workspace that contains the EclipseLInk development Eclipse projects and alter the persistence.xml to point at your DB.
Comment 4 Tom Ware CLA 2010-11-18 11:16:38 EST 
FYI: recreation has been attached for informational purposes only, a recreation of this issue that does show other issues as well is still required.
Comment 5 Eclipse Webmaster CLA 2022-06-09 10:10:09 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink