This site is maintained for archival purposes only. Eclipse projects have transitioned to GitHub and Eclipse GitLab. Use the Projects search tool to locate your project and access its latest code and issue tracker.
Bug 218084 - Implement metadata merging functionality between mapping files
Summary: Implement metadata merging functionality between mapping files
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: Guy Pelletier CLA
QA Contact:
URL:
Whiteboard: Fixed in 1.0M8
Keywords:
Depends on:
Blocks: 200040
  Show dependency tree
 
Reported: 2008-02-06 16:28 EST by Guy Pelletier CLA
Modified: 2022-06-09 10:16 EDT (History)
2 users (show)

See Also:

Attachments
Proposed patch (846.12 KB, patch)
2008-05-16 13:29 EDT, Guy Pelletier CLA 		no flags Details | Diff
Updated patch (848.02 KB, patch)
2008-05-16 15:04 EDT, Guy Pelletier CLA 		no flags Details | Diff
Final patch (1.08 MB, patch)
2008-05-20 11:36 EDT, Guy Pelletier 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 Guy Pelletier CLA 2008-02-06 16:28:11 EST 
Currently, orm mapping files do not merge their metadata with other mapping files. 

That is, entity, embeddable etc. definitions are assumed to be fully defined within a mapping file. If an entity is defined in multiple files, results are unexpected as a 'last in win' approach is currently in place.
Comment 1 Guy Pelletier CLA 2008-04-17 14:03:37 EDT 
See the functional spec for the details of this feature.

http://wiki.eclipse.org/EclipseLink/Development/200040/FunctionalSpec#Override_and_Merging_Rules
Comment 2 Guy Pelletier CLA 2008-05-16 13:29:28 EDT 
Created attachment 100683 [details]
Proposed patch
Comment 3 Guy Pelletier CLA 2008-05-16 15:04:57 EDT 
Created attachment 100706 [details]
Updated patch
Comment 4 Guy Pelletier CLA 2008-05-20 11:36:03 EDT 
Created attachment 101074 [details]
Final patch
Comment 5 Guy Pelletier CLA 2008-05-20 14:56:49 EDT 
The patch has been submitted.

Reviewed by: Tom Ware

No new tests were added. However, the existing xml extended advanced model 
was modified to include an eclipselink-orm.xml file with the extended 
mappings for that model. Hence reverting the 
advanced-extended-entity-mappings.xml for that model back to its original 
JPA format.

All tests (JPA and extended) should continue to pass without any errors.
Comment 6 Liza Rekadze CLA 2008-05-30 14:29:14 EDT 
(In reply to comment #5)

Should this be documented in EclipseLink Dev. Guide?
Comment 7 Guy Pelletier CLA 2008-05-30 14:48:40 EDT 
Yes. I have entered a new bug 234914 for this.
Comment 8 Eclipse Webmaster CLA 2022-06-09 10:09:02 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink
Comment 9 Eclipse Webmaster CLA 2022-06-09 10:16:08 EDT 
The Eclipselink project has moved to Github: https://github.com/eclipse-ee4j/eclipselink