This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
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

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