Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 328650 - buildReferencesPKList creates incorrect array for MapKey relationship
Summary: buildReferencesPKList creates incorrect array for MapKey relationship
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: 2010-10-25 15:48 EDT by Gordon Yorke CLA
Modified: 2022-06-09 10:34 EDT (History)
2 users (show)

See Also:

Attachments
Simple Patch and Test Case (4.13 KB, patch)
2010-10-27 11:50 EDT, Gordon Yorke CLA 		no flags Details | Diff
patch changes missed from the first file (822 bytes, patch)
2010-10-27 13:54 EDT, Gordon Yorke CLA 		no flags Details | Diff
Patch combines previous patches into one file and adds a small change (4.95 KB, patch)
2010-10-27 16:20 EDT, Gordon Yorke CLA 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Gordon Yorke CLA 2010-10-25 15:48:33 EDT 
buildReferencesPKList creates an array of size collection.size()*2 when processing a MapKey relationship.  That array size is needed for MapKeyColumns but not for MapKey where the Key is derived from the value.
Comment 1 Gordon Yorke CLA 2010-10-27 11:50:37 EDT 
Created attachment 181850 [details]
Simple Patch and Test Case

Simply update to change the size of the Object[] created by the mapping when preparing the PK list of the map.  In the case of the MapKey the size should equal the size of the map.
Comment 2 Gordon Yorke CLA 2010-10-27 13:21:00 EDT 
Patch Checked in
Comment 3 Gordon Yorke CLA 2010-10-27 13:54:22 EDT 
Created attachment 181856 [details]
patch changes missed from the first file

Part of the change was missing from that first patch file.
Comment 4 Gordon Yorke CLA 2010-10-27 16:20:24 EDT 
Created attachment 181879 [details]
Patch combines previous patches into one file and adds a small change
Comment 5 Chris Delahunt CLA 2010-11-02 15:05:26 EDT 
Patch also checked into EclipseLink 2.1.2 revision 8452
Comment 6 Eclipse Webmaster CLA 2022-06-09 10:14:10 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 7 Eclipse Webmaster CLA 2022-06-09 10:34:47 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink