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

Bug 348273

Summary: Reduce the memory footprint of XMLDescriptor
Product: z_Archived Reporter: Blaise Doughan <blaise.doughan>
Component: EclipselinkAssignee: Blaise Doughan <blaise.doughan>
Status: CLOSED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: eclipselink.oxm-inbox
Version: unspecified   
Target Milestone: ---   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on:    
Bug Blocks: 348272, 348465    
Attachments:
Description Flags
Core - Fix
none
Core - Fix (2.1.4)
none
Core - Fix (2.2.1)
none
Core - Fix (Trunk)
none
MOXy - Test Cases
none
Core - Fix (2.1.4) - Part 2
none
Core - Fix (2.2.1) - Part 2
none
Core - Fix (Trunk) - Part 2 none

Description Blaise Doughan CLA 2011-06-03 15:38:06 EDT

    
Comment 1 Blaise Doughan CLA 2011-06-03 15:40:35 EDT
Created attachment 197326 [details]
Core - Fix
Comment 2 Blaise Doughan CLA 2011-06-06 11:01:59 EDT
Created attachment 197403 [details]
Core - Fix (2.1.4)
Comment 3 Blaise Doughan CLA 2011-06-06 11:07:08 EDT
Created attachment 197405 [details]
Core - Fix (2.2.1)
Comment 4 Blaise Doughan CLA 2011-06-06 15:44:08 EDT
Created attachment 197451 [details]
Core - Fix (Trunk)
Comment 5 Blaise Doughan CLA 2011-06-06 15:44:23 EDT
Created attachment 197452 [details]
MOXy - Test Cases
Comment 6 Blaise Doughan CLA 2011-06-06 16:28:25 EDT
Fix checked into 2.1.4 at rev: 9521
Fix checked into 2.2.1 at rev: 9522
Fix checked into trunk at rev: 9523

The following bug has been opened for the 2.3.1 stream:
https://bugs.eclipse.org/348465

Code reviewed by:  Rick Barkhouse & Gordon Yorke

Fix Description:
- Added a hasEventManager method to ClassDescriptor that checks if an event manager has been set.  This has been done since the getEventManager() method will init one.
- Changed code to use the hasEventManager method.
- Changed XMLDirectMapping so it does not trigger a CopyPolicy to be created.
Comment 7 Blaise Doughan CLA 2011-06-07 12:56:31 EDT
This fix is causing core and OXM test failures.
Comment 8 Blaise Doughan CLA 2011-06-07 13:30:37 EDT
Created attachment 197524 [details]
Core - Fix (2.1.4) - Part 2
Comment 9 Blaise Doughan CLA 2011-06-07 13:30:58 EDT
Created attachment 197525 [details]
Core - Fix (2.2.1) - Part 2
Comment 10 Blaise Doughan CLA 2011-06-07 13:31:33 EDT
Created attachment 197526 [details]
Core - Fix (Trunk) - Part 2
Comment 11 Blaise Doughan CLA 2011-06-07 14:37:17 EDT
Part 2 checked into 2.1.4 at rev:  9533
Part 2 checked into 2.2.1 at rev:  9534
Part 2 checked into trunk at rev:  9535

Code reviewed by:  Matt MacIvor

Updated fix for:
Bug 348273 - Reduce the memory footprint of XMLDescriptor 

Now if the parent descriptor has an event manager, the child descriptor gets an event manager.
Comment 12 Eclipse Webmaster CLA 2022-06-09 10:09:59 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink