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

Bug 319403

Summary: Thread safety issue with attachment handling
Product: z_Archived Reporter: Blaise Doughan <blaise.doughan>
Component: EclipselinkAssignee: Nobody - feel free to take it <nobody>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eclipselink.sdo-inbox
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Attachments:
Description Flags
Core - Fix
none
SDO - Fix
none
SDO - Test Cases none

Description Blaise Doughan CLA 2010-07-09 11:17:49 EDT
There is an issue where a save can add too many attachments, or a load can have too few attachments.
Comment 1 Blaise Doughan CLA 2010-07-09 11:29:43 EDT
Created attachment 173872 [details]
Core - Fix

Added clone methods to XMLMarshaller & XMLUnmarshaller
Comment 2 Blaise Doughan CLA 2010-07-09 11:31:54 EDT
Created attachment 173874 [details]
SDO - Fix

- Added XMLHelper load options for attachment marshaller/unmarshaller
- Always get a new marshaller/unmarshaller (a clone of the one previously used)
- Options override anything set on the template unmarshaller (getXmlMarshaller()/getXmlUnmarshaller())
Comment 3 Blaise Doughan CLA 2010-07-09 13:23:53 EDT
Created attachment 173883 [details]
SDO - Test Cases
Comment 4 Blaise Doughan CLA 2010-07-12 10:17:59 EDT
Fix checked into 1.2.1 at rev:  7785
Fix checked into 2.0.3 at rev:  7786
Fix checked into 2.1.1 at rev:  7787
Fix checked into trunk at rev:  7788

Code reviewed by Matt MacIvor
Comment 5 Eclipse Webmaster CLA 2022-06-09 10:29:56 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink