Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 336352 - Memory Improvement: Share Descriptors for Built-In Types
Summary: Memory Improvement: Share Descriptors for Built-In Types
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: David McCann CLA
QA Contact:
URL:
Whiteboard:
Keywords:
: 336962 (view as bug list)
Depends on: 334284
Blocks: 334389 337601
  Show dependency tree
 
Reported: 2011-02-04 10:02 EST by Blaise Doughan CLA
Modified: 2022-06-09 10:10 EDT (History)
1 user (show)

See Also:


Attachments
Proposed fix. (17.68 KB, patch)
2011-02-11 11:55 EST, David McCann CLA
no flags Details | Diff
SDO - Fix (16.38 KB, patch)
2011-02-17 13:20 EST, Blaise Doughan CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Blaise Doughan CLA 2011-02-04 10:02:21 EST
Currently each SDOHelperContext will have its own XMLDescriptor for each of the built in types.  To improve memory usage these descriptors should be shared between SDOHelperContexts.
Comment 1 Blaise Doughan CLA 2011-02-04 10:05:12 EST
Implementation Note:

Currently when new types are added SDO may modify the descriptor to add new default root elements.  Instead this could be handled by directly modifying the QName maps on XMLContext instead.
Comment 2 David McCann CLA 2011-02-11 11:55:39 EST
Created attachment 188794 [details]
Proposed fix.
Comment 3 David McCann CLA 2011-02-11 11:55:55 EST
*** Bug 336962 has been marked as a duplicate of this bug. ***
Comment 4 Blaise Doughan CLA 2011-02-17 13:20:20 EST
Created attachment 189212 [details]
SDO - Fix
Comment 5 David McCann CLA 2011-02-18 12:25:54 EST
2.1.3 - Fix checked in
Reviewed by:  blaise.doughan@oracle.com
Tests: all unit tests pass as expected
Revision: 8992
Comment 6 David McCann CLA 2011-02-18 13:11:15 EST
2.3 - Fix checked in
Reviewed by:  blaise.doughan@oracle.com
Tests: all unit tests pass as expected
Revision: 8993
Comment 7 Blaise Doughan CLA 2011-02-18 13:56:25 EST
The following bug has been entered to have this fix applied to the 2.2.1 stream:

https://bugs.eclipse.org/bugs/show_bug.cgi?id=337601
Comment 8 Eclipse Webmaster CLA 2022-06-09 10:10:41 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink