Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 369273 - Foundation should cache CacheKey on enhanced entity to prevent double cache lookup when cloning relationships
Summary: Foundation should cache CacheKey on enhanced entity to prevent double cache l...
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Eclipselink (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Nobody - feel free to take it CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-01-20 13:54 EST by Gordon Yorke CLA
Modified: 2022-06-09 10:20 EDT (History)
1 user (show)

See Also:


Attachments
proposed Patch (24.32 KB, patch)
2012-01-24 13:37 EST, Gordon Yorke CLA
no flags Details | Diff
Final Patch (24.30 KB, patch)
2012-01-24 14:36 EST, Gordon Yorke CLA
no flags Details | Diff
additional update (4.02 KB, patch)
2012-01-31 15:02 EST, Gordon Yorke CLA
no flags Details | Diff
Update to check for null (3.84 KB, patch)
2012-02-01 11:00 EST, Gordon Yorke CLA
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Yorke CLA 2012-01-20 13:54:27 EST
When EclipseLink clones a relationship into a UnitOfWork for each element in that relationship the cache will be accessed twice.  By caching the cacheKey on the enhanced entity we can prevent subsequent unneeded lookups.
Comment 1 Gordon Yorke CLA 2012-01-24 13:37:45 EST
Created attachment 209992 [details]
proposed Patch
Comment 2 Gordon Yorke CLA 2012-01-24 14:36:49 EST
Created attachment 209999 [details]
Final Patch
Comment 3 Gordon Yorke CLA 2012-01-25 12:19:30 EST
Patch checked in
Reviewed by James Sutherland
Comment 4 Gordon Yorke CLA 2012-01-31 15:02:19 EST
Created attachment 210332 [details]
additional update
Comment 5 Gordon Yorke CLA 2012-02-01 11:00:35 EST
Created attachment 210379 [details]
Update to check for null
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:16:23 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:20:51 EDT
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink