fix for bug 217879

View: New views
20 Messages — Rating Filter:   Alert me  
< Prev | 1 - 2 - 3 - 4 - 5 - 6 - 7 | Next >

Re: fix for bug 211302

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Changes have been submitted.

Reviewed by: Gordon Yorke

New test suite added:
org.eclipse.persistence.testing.tests.jpa.relationships.RelationshipModelJUnitTestSuite

New tests added to:
org.eclipse.persistence.testing.tests.jpa.xml.relationships.EntityMappingsRelationshipsJUnitTestCase
- testCreateExtendedItem
- testModifyExtendedItem
- testVerifyExtendedItem

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Wednesday, March 26, 2008 10:14 AM
Subject: [eclipselink-dev] fix for bug 211302


>A patch file for bug 211302 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=211302
>
> Description: Add variable 1-1 mapping support to the EclipseLink-ORM.XML
> Schema
>
> Please review and provide any feedback.
>
> Cheers,
> Guy
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Testing change

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Some parts of this message have been removed. Learn more about Nabble's security policy.
The testing framework has now been modified to allow proper testing of the extended Eclipselink configurations from XML.
 
You now need to set a system property to run the extended XML tests. The property is:
 
 -Dorm.testing=jpa (default), or
 -Dorm.testing=eclipselink
 
Also note, that if testing the extended configurations you must only have the eclipselink-xml-extended-model.jar on the classpath.  On the flip side, for jpa testing do not include this jar on the classpath.
 
The classes in the extended jar are available from the other testing jars. Therefore, including all the jars together on the classpath will cause unexpected results and errors since individual classes can only be weaved once (within the context of the first jar file where it is processed).
 
Cheers,
Guy
 

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

fix for bug 224155

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

A patch file for bug 224155 is now available for review.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=224155

Description: embeddable-attributes should be extended in the EclipseLink
ORM.XML schema

Please review and provide any feedback.

Cheers,
Guy

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: fix for bug 224155

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Changes have been submitted.

Reviewed: Tom Ware

New test suite added:
org.eclipse.persistence.testing.tests.jpa.xml.complexaggregate.EntityMappingsComplexAggregateJUnitTestCase

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Tuesday, April 08, 2008 2:37 PM
Subject: [eclipselink-dev] fix for bug 224155


>A patch file for bug 224155 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=224155
>
> Description: embeddable-attributes should be extended in the EclipseLink
> ORM.XML schema
>
> Please review and provide any feedback.
>
> Cheers,
> Guy
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

check in for fix for bug 221226

by Peter Krogh :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

These three related bugs are fixed by this transaction.

221226 Support a provided classloader in EclipseLink JPA EntityManagerFactory Creation
        214769 DatasourceLogin.setPlatformClassName should take optional ClassLoader
        221225 EntityManagerSetupImpl should set PersistenceUnitInfo classloader on DatasourceLogin

Patch is on this bug - https://bugs.eclipse.org/bugs/show_bug.cgi?id=221226

This fix was written by Shaun Smith and Peter Krogh, and reviewed by Tom Ware.

New API:
        221226 adds a property called eclipselink.classloader.  
        /*
         * Specify the classloader to use to create an EntityManagerFactory in the
       * property map passed to Persistence.createEntityManagerFactory.
       * This is a dynamic property, that can only be set at runtime in the property
       * map, and is not for use in the persistence.xml
       */


_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Proporal for bug 211326

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

A proposal for bug 211326 (Add default-timeout support to the
EclipseLink-ORM.XML Schema) has been put forth.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=211326

Please review and provide any feedback.

Cheers,
Guy

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
review

Thanks,

Andrei

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

RE: Proposed patch for bug 217168

by Peter Krogh :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Thanks Andrei.  Minor thing, can you put the description of the bug in the email?  I like to beable to search and keep track of changes by looking through my inbox

-----Original Message-----
From: eclipselink-dev-bounces@...
[mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
Sent: Wednesday, April 09, 2008 5:18 PM
To: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] Proposed patch for bug 217168


Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
review

Thanks,

Andrei

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Bug 217168: Add mapping's properties support to the EclipseLink-ORM.XML
Schema
----- Original Message -----
From: "Peter Krogh" <peter.krogh@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Thursday, April 10, 2008 8:43 AM
Subject: RE: [eclipselink-dev] Proposed patch for bug 217168


Thanks Andrei.  Minor thing, can you put the description of the bug in the
email?  I like to beable to search and keep track of changes by looking
through my inbox

-----Original Message-----
From: eclipselink-dev-bounces@...
[mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
Sent: Wednesday, April 09, 2008 5:18 PM
To: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] Proposed patch for bug 217168


Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
review

Thanks,

Andrei

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

It's: "Add mapping's properties support to the EclipseLink-ORM.XML Schema"
----- Original Message -----
From: "Peter Krogh" <peter.krogh@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Thursday, April 10, 2008 8:43 AM
Subject: RE: [eclipselink-dev] Proposed patch for bug 217168


Thanks Andrei.  Minor thing, can you put the description of the bug in the
email?  I like to beable to search and keep track of changes by looking
through my inbox

-----Original Message-----
From: eclipselink-dev-bounces@...
[mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
Sent: Wednesday, April 09, 2008 5:18 PM
To: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] Proposed patch for bug 217168


Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
review

Thanks,

Andrei

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Parent Message unknown Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

The new patch (containing tests) is ready for review. Please disregard the
previous one.

Thanks,

Andrei

----- Original Message -----
From: "Andrei Ilitchev" <andrei.ilitchev@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Thursday, April 10, 2008 9:06 AM
Subject: Re: [eclipselink-dev] Proposed patch for bug 217168


> It's: "Add mapping's properties support to the EclipseLink-ORM.XML Schema"
> ----- Original Message -----
> From: "Peter Krogh" <peter.krogh@...>
> To: "Dev mailing list for Eclipse Persistence Services"
> <eclipselink-dev@...>
> Sent: Thursday, April 10, 2008 8:43 AM
> Subject: RE: [eclipselink-dev] Proposed patch for bug 217168
>
>
> Thanks Andrei.  Minor thing, can you put the description of the bug in the
> email?  I like to beable to search and keep track of changes by looking
> through my inbox
>
> -----Original Message-----
> From: eclipselink-dev-bounces@...
> [mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
> Sent: Wednesday, April 09, 2008 5:18 PM
> To: Dev mailing list for Eclipse Persistence Services
> Subject: [eclipselink-dev] Proposed patch for bug 217168
>
>
> Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
> review
>
> Thanks,
>
> Andrei
>
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Parent Message unknown Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

A new patch was created after Guy's review of the prev. patch.

----- Original Message -----
From: "Andrei Ilitchev" <andrei.ilitchev@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Friday, April 11, 2008 4:36 PM
Subject: Re: [eclipselink-dev] Proposed patch for bug 217168


> The new patch (containing tests) is ready for review. Please disregard the
> previous one.
>
> Thanks,
>
> Andrei
>
> ----- Original Message -----
> From: "Andrei Ilitchev" <andrei.ilitchev@...>
> To: "Dev mailing list for Eclipse Persistence Services"
> <eclipselink-dev@...>
> Sent: Thursday, April 10, 2008 9:06 AM
> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>
>
>> It's: "Add mapping's properties support to the EclipseLink-ORM.XML
>> Schema"
>> ----- Original Message -----
>> From: "Peter Krogh" <peter.krogh@...>
>> To: "Dev mailing list for Eclipse Persistence Services"
>> <eclipselink-dev@...>
>> Sent: Thursday, April 10, 2008 8:43 AM
>> Subject: RE: [eclipselink-dev] Proposed patch for bug 217168
>>
>>
>> Thanks Andrei.  Minor thing, can you put the description of the bug in
>> the email?  I like to beable to search and keep track of changes by
>> looking through my inbox
>>
>> -----Original Message-----
>> From: eclipselink-dev-bounces@...
>> [mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
>> Sent: Wednesday, April 09, 2008 5:18 PM
>> To: Dev mailing list for Eclipse Persistence Services
>> Subject: [eclipselink-dev] Proposed patch for bug 217168
>>
>>
>> Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
>> review
>>
>> Thanks,
>>
>> Andrei
>>
>> _______________________________________________
>> eclipselink-dev mailing list
>> eclipselink-dev@...
>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>
>> _______________________________________________
>> eclipselink-dev mailing list
>> eclipselink-dev@...
>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

fix for bug 226517

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

A patch file for bug 226517 is now available for review.
 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=226517
 
 Description: Add existence support to the EclipseLink-ORM.XML Schema
 
 Please review and provide any feedback.
 
 Cheers,
Guy
_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Parent Message unknown Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Yet another patch for this bug is ready to be reviewed - see details in the
bug.
----- Original Message -----
From: "Andrei Ilitchev" <andrei.ilitchev@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Monday, April 14, 2008 4:19 PM
Subject: Re: [eclipselink-dev] Proposed patch for bug 217168


>A new patch was created after Guy's review of the prev. patch.
>
> ----- Original Message -----
> From: "Andrei Ilitchev" <andrei.ilitchev@...>
> To: "Dev mailing list for Eclipse Persistence Services"
> <eclipselink-dev@...>
> Sent: Friday, April 11, 2008 4:36 PM
> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>
>
>> The new patch (containing tests) is ready for review. Please disregard
>> the previous one.
>>
>> Thanks,
>>
>> Andrei
>>
>> ----- Original Message -----
>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>> To: "Dev mailing list for Eclipse Persistence Services"
>> <eclipselink-dev@...>
>> Sent: Thursday, April 10, 2008 9:06 AM
>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>
>>
>>> It's: "Add mapping's properties support to the EclipseLink-ORM.XML
>>> Schema"
>>> ----- Original Message -----
>>> From: "Peter Krogh" <peter.krogh@...>
>>> To: "Dev mailing list for Eclipse Persistence Services"
>>> <eclipselink-dev@...>
>>> Sent: Thursday, April 10, 2008 8:43 AM
>>> Subject: RE: [eclipselink-dev] Proposed patch for bug 217168
>>>
>>>
>>> Thanks Andrei.  Minor thing, can you put the description of the bug in
>>> the email?  I like to beable to search and keep track of changes by
>>> looking through my inbox
>>>
>>> -----Original Message-----
>>> From: eclipselink-dev-bounces@...
>>> [mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei Ilitchev
>>> Sent: Wednesday, April 09, 2008 5:18 PM
>>> To: Dev mailing list for Eclipse Persistence Services
>>> Subject: [eclipselink-dev] Proposed patch for bug 217168
>>>
>>>
>>> Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
>>> review
>>>
>>> Thanks,
>>>
>>> Andrei
>>>
>>> _______________________________________________
>>> eclipselink-dev mailing list
>>> eclipselink-dev@...
>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>
>>> _______________________________________________
>>> eclipselink-dev mailing list
>>> eclipselink-dev@...
>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>
>>
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: fix for bug 226517

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Changes have been submitted.

Reviewed by: Tom Ware

New test (testExistenceCheckingSetting) added to 3 suites:
org.eclipse.persistence.testing.tests.jpa.advanced.AdvancedJPAJunitTest
org.eclipse.persistence.testing.tests.jpa.inherited.EmbeddableSuperclassJunitTest
org.eclipse.persistence.testing.tests.jpa.xml.advanced.EntityMappingsAdvancedJUnitTestCase

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Tuesday, April 15, 2008 11:02 AM
Subject: [eclipselink-dev] fix for bug 226517


>A patch file for bug 226517 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=226517
>
> Description: Add existence support to the EclipseLink-ORM.XML Schema
>
> Please review and provide any feedback.
>
> Cheers,
> Guy _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Parent Message unknown Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

The latest patch is ready for review - exception now thrown in case two
same-named properties
applied to the same master in either orm xml or annotations.
----- Original Message -----
From: "Andrei Ilitchev" <andrei.ilitchev@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Tuesday, April 15, 2008 2:20 PM
Subject: Re: [eclipselink-dev] Proposed patch for bug 217168


> Yet another patch for this bug is ready to be reviewed - see details in
> the bug.
> ----- Original Message -----
> From: "Andrei Ilitchev" <andrei.ilitchev@...>
> To: "Dev mailing list for Eclipse Persistence Services"
> <eclipselink-dev@...>
> Sent: Monday, April 14, 2008 4:19 PM
> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>
>
>>A new patch was created after Guy's review of the prev. patch.
>>
>> ----- Original Message -----
>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>> To: "Dev mailing list for Eclipse Persistence Services"
>> <eclipselink-dev@...>
>> Sent: Friday, April 11, 2008 4:36 PM
>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>
>>
>>> The new patch (containing tests) is ready for review. Please disregard
>>> the previous one.
>>>
>>> Thanks,
>>>
>>> Andrei
>>>
>>> ----- Original Message -----
>>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>>> To: "Dev mailing list for Eclipse Persistence Services"
>>> <eclipselink-dev@...>
>>> Sent: Thursday, April 10, 2008 9:06 AM
>>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>>
>>>
>>>> It's: "Add mapping's properties support to the EclipseLink-ORM.XML
>>>> Schema"
>>>> ----- Original Message -----
>>>> From: "Peter Krogh" <peter.krogh@...>
>>>> To: "Dev mailing list for Eclipse Persistence Services"
>>>> <eclipselink-dev@...>
>>>> Sent: Thursday, April 10, 2008 8:43 AM
>>>> Subject: RE: [eclipselink-dev] Proposed patch for bug 217168
>>>>
>>>>
>>>> Thanks Andrei.  Minor thing, can you put the description of the bug in
>>>> the email?  I like to beable to search and keep track of changes by
>>>> looking through my inbox
>>>>
>>>> -----Original Message-----
>>>> From: eclipselink-dev-bounces@...
>>>> [mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei
>>>> Ilitchev
>>>> Sent: Wednesday, April 09, 2008 5:18 PM
>>>> To: Dev mailing list for Eclipse Persistence Services
>>>> Subject: [eclipselink-dev] Proposed patch for bug 217168
>>>>
>>>>
>>>> Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 for
>>>> review
>>>>
>>>> Thanks,
>>>>
>>>> Andrei
>>>>
>>>> _______________________________________________
>>>> eclipselink-dev mailing list
>>>> eclipselink-dev@...
>>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>>
>>>> _______________________________________________
>>>> eclipselink-dev mailing list
>>>> eclipselink-dev@...
>>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>>
>>>
>>
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: Proposed patch for bug 217168

by Andrei Ilitchev :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Checked in the latest patch.
Reviewed by Guy.
Tests are in FullRegressionTestSuite:
for annotations it's AdvancedJPAJunitTest.testProperty for annotations and
EntityMappingsAdvancedJUnitTestCase.testProperty for xml (for the latter
specify -Dorm.testing=eclipselink)
----- Original Message -----
From: "Andrei Ilitchev" <andrei.ilitchev@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Wednesday, April 16, 2008 11:00 AM
Subject: Re: [eclipselink-dev] Proposed patch for bug 217168


> The latest patch is ready for review - exception now thrown in case two
> same-named properties
> applied to the same master in either orm xml or annotations.
> ----- Original Message -----
> From: "Andrei Ilitchev" <andrei.ilitchev@...>
> To: "Dev mailing list for Eclipse Persistence Services"
> <eclipselink-dev@...>
> Sent: Tuesday, April 15, 2008 2:20 PM
> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>
>
>> Yet another patch for this bug is ready to be reviewed - see details in
>> the bug.
>> ----- Original Message -----
>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>> To: "Dev mailing list for Eclipse Persistence Services"
>> <eclipselink-dev@...>
>> Sent: Monday, April 14, 2008 4:19 PM
>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>
>>
>>>A new patch was created after Guy's review of the prev. patch.
>>>
>>> ----- Original Message -----
>>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>>> To: "Dev mailing list for Eclipse Persistence Services"
>>> <eclipselink-dev@...>
>>> Sent: Friday, April 11, 2008 4:36 PM
>>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>>
>>>
>>>> The new patch (containing tests) is ready for review. Please disregard
>>>> the previous one.
>>>>
>>>> Thanks,
>>>>
>>>> Andrei
>>>>
>>>> ----- Original Message -----
>>>> From: "Andrei Ilitchev" <andrei.ilitchev@...>
>>>> To: "Dev mailing list for Eclipse Persistence Services"
>>>> <eclipselink-dev@...>
>>>> Sent: Thursday, April 10, 2008 9:06 AM
>>>> Subject: Re: [eclipselink-dev] Proposed patch for bug 217168
>>>>
>>>>
>>>>> It's: "Add mapping's properties support to the EclipseLink-ORM.XML
>>>>> Schema"
>>>>> ----- Original Message -----
>>>>> From: "Peter Krogh" <peter.krogh@...>
>>>>> To: "Dev mailing list for Eclipse Persistence Services"
>>>>> <eclipselink-dev@...>
>>>>> Sent: Thursday, April 10, 2008 8:43 AM
>>>>> Subject: RE: [eclipselink-dev] Proposed patch for bug 217168
>>>>>
>>>>>
>>>>> Thanks Andrei.  Minor thing, can you put the description of the bug in
>>>>> the email?  I like to beable to search and keep track of changes by
>>>>> looking through my inbox
>>>>>
>>>>> -----Original Message-----
>>>>> From: eclipselink-dev-bounces@...
>>>>> [mailto:eclipselink-dev-bounces@...]On Behalf Of Andrei
>>>>> Ilitchev
>>>>> Sent: Wednesday, April 09, 2008 5:18 PM
>>>>> To: Dev mailing list for Eclipse Persistence Services
>>>>> Subject: [eclipselink-dev] Proposed patch for bug 217168
>>>>>
>>>>>
>>>>> Added patch to https://bugs.eclipse.org/bugs/show_bug.cgi?id=217168 
>>>>> for
>>>>> review
>>>>>
>>>>> Thanks,
>>>>>
>>>>> Andrei
>>>>>
>>>>> _______________________________________________
>>>>> eclipselink-dev mailing list
>>>>> eclipselink-dev@...
>>>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>>>
>>>>> _______________________________________________
>>>>> eclipselink-dev mailing list
>>>>> eclipselink-dev@...
>>>>> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>>>>>
>>>>
>>>
>>
>
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Bug 218084

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Work is about to begin to resolve bug 218084 - Implement metadata merging
functionality between mapping files

As of now, the implementation will align with the details provided in the
functional spec:

http://wiki.eclipse.org/EclipseLink/Development/200040/FunctionalSpec#Override_and_Merging_Rules

Therefore, this is a last call for feedback. The implementation of this
feature is slated to begin on Monday, April 21st assuming there are no major
issues raised in the feedback.

Please provide any comments you may have about this feature in the bug:
https://bugs.eclipse.org/bugs/show_bug.cgi?id=218084

Thanks,
Guy


_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

Re: Bug 218084 - XML merging

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

The work for this feature has now been completed. However another issue came
up during testing.

Within our extended tests we have 3 persistence unit (Employee, Relationship
and Complex Aggregate). After creating the eclipselink-orm.xml which
contained extensions for the Employee persistence unit only, validation
exceptions would occur for the other persistence units since they would
automatically pick up the eclipselink-orm file (just like an orm.xml file
would be).

To get around the problem, I introduced the following property that can be
use in the persistence.xml to turn off the automatic discovering of the
eclipselink-orm.xml file.
<property name="eclipselink.exclude-eclipselink-orm" value="true"/>

So the question is, do we like this solution?

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Thursday, April 17, 2008 2:17 PM
Subject: [eclipselink-dev] Bug 218084


> Work is about to begin to resolve bug 218084 - Implement metadata merging
> functionality between mapping files
>
> As of now, the implementation will align with the details provided in the
> functional spec:
>
> http://wiki.eclipse.org/EclipseLink/Development/200040/FunctionalSpec#Override_and_Merging_Rules
>
> Therefore, this is a last call for feedback. The implementation of this
> feature is slated to begin on Monday, April 21st assuming there are no
> major issues raised in the feedback.
>
> Please provide any comments you may have about this feature in the bug:
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=218084
>
> Thanks,
> Guy
>
>
> _______________________________________________
> eclipselink-dev mailing list
> eclipselink-dev@...
> https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
>

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

fix for bug 218084

by Guy Pelletier :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

A patch file for bug 218084 is now available for review.

https://bugs.eclipse.org/bugs/show_bug.cgi?id=218084

 Description: Implement metadata merging functionality between mapping files

 Please review and provide any feedback.

Cheers,
Guy

_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev
< Prev | 1 - 2 - 3 - 4 - 5 - 6 - 7 | Next >