fix for bug 217879

[Guy Pelletier]

A patch file for bug 217879 is now available for review.
 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=217879

Description:
 Implement EntityCaching sections of bug 211298
 
 Please review and provide any feedback.
 
Cheers,
Guy
_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

[Guy Pelletier]

Changes have been submitted.

Reviewed by Gordon Yorke

New test added to
org.eclipse.persistence.testing.tests.jpa.xml.advanced.EntityMappingsAdvancedJUnitTestCase:
- testEmployeeCacheSettings

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Sunday, February 24, 2008 7:19 AM
Subject: [eclipselink-dev] fix for bug 217879

>A patch file for bug 217879 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=217879
>
> Description: Implement EntityCaching sections of bug 211298
>
> 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

[Guy Pelletier]

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

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

Description: Implement Change tracking and Descriptor customizer sections of
bug 211298

Please review and provide any feedback.

Cheers,
Guy

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

[Guy Pelletier]

Changes have been submitted.

Reviewed: Tom Ware

New tests added to
org.eclipse.persistence.testing.tests.jpa.xml.advanced.EntityMappingsAdvancedJUnitTestCase
 - testReadOnlyClassSetting
 - testEmployeeChangeTrackingPolicy
 - testAddressChangeTrackingPolicy
 - testPhoneNumberChangeTrackingPolicy
 - testProjectChangeTrackingPolicy
 - testEmployeeCacheSettings (modified to test a customizer class)

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Wednesday, February 27, 2008 3:40 PM
Subject: [eclipselink-dev] fix for bug 217880

>A patch file for bug 217880 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=217880
>
> Description: Implement Change tracking and Descriptor customizer sections
> of bug 211298
>
> 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

[Guy Pelletier]

A patch file for bug 217883 is now available for review.
 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=217883

Description: Implement Stored procedure query sections of bug 211298
 
 Please review and provide any feedback.
 
Cheers,
Guy
_______________________________________________
eclipselink-dev mailing list
eclipselink-dev@...
https://dev.eclipse.org/mailman/listinfo/eclipselink-dev

[Guy Pelletier]

Changes have been submitted.

Reviewed: Tom Ware

New tests added to
org.eclipse.persistence.testing.tests.jpa.xml.advanced.EntityMappingsAdvancedJUnitTestCase
 - testNamedStoredProcedureQuery
 - testNamedStoredProcedureQueryInOut

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Friday, February 29, 2008 3:41 PM
Subject: [eclipselink-dev] fix for bug 217883

>A patch file for bug 217883 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=217883
>
> Description: Implement Stored procedure query sections of bug 211298
>
> 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

[Guy Pelletier]

A proposal has been documented for the following bug:

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.

Thanks,
Guy

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

[JAMES.SUTHERLAND]

Has the design document been updated to include the design for this feature?  It would seem that adding something as complex as interface descriptors into JPA would require some design, since this also involves implications to the runtime and querying.  Note that the current proposal in the bug log will not work.

http://wiki.eclipse.org/EclipseLink/Development/1.0/ER-200040_DS_M4

The above design document does not seem to have any information on this feature yet.


-----Original Message-----
From: eclipselink-dev-bounces@... [mailto:eclipselink-dev-bounces@...]On Behalf Of Guy Pelletier
Sent: Tuesday, March 04, 2008 11:47 AM
To: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] fix for bug 211302

A proposal has been documented for the following bug:

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.

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

[Gordon Yorke-2]

Very clearly this feature is not about exposing Interface Descriptor support through JPA.  This feature is simply about adding mapping support to the EclipseLink-ORM.xml schema.  Post this feature completion there will be work (and another feature) involved in exposing query support (including EntityManager updates) for interfaces etc...

Perhaps you can shed some detail on why the current proposal will not work for adding interface mapping support to EclipseLink?

--Gordon

-----Original Message-----
From: eclipselink-dev-bounces@...
[mailto:eclipselink-dev-bounces@...]On Behalf Of James
Sutherland
Sent: Wednesday, March 05, 2008 10:39 AM
To: Guy Pelletier; Dev mailing list for Eclipse Persistence Services
Subject: RE: [eclipselink-dev] fix for bug 211302


Has the design document been updated to include the design for this feature?  It would seem that adding something as complex as interface descriptors into JPA would require some design, since this also involves implications to the runtime and querying.  Note that the current proposal in the bug log will not work.

http://wiki.eclipse.org/EclipseLink/Development/1.0/ER-200040_DS_M4

The above design document does not seem to have any information on this feature yet.


-----Original Message-----
From: eclipselink-dev-bounces@... [mailto:eclipselink-dev-bounces@...]On Behalf Of Guy Pelletier
Sent: Tuesday, March 04, 2008 11:47 AM
To: Dev mailing list for Eclipse Persistence Services
Subject: [eclipselink-dev] fix for bug 211302

A proposal has been documented for the following bug:

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.

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

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

[Guy Pelletier]

A patch file for bug 221658 is now available for review.
 
https://bugs.eclipse.org/bugs/show_bug.cgi?id=221658

Description: Metadata processing model re-work and clean-up
 
 Please review and provide any feedback.
 
Cheers,
Guy

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

[Guy Pelletier]

Changes have been submitted.

Reviewed: Tom Ware

New tests added to:
org.eclipse.persistence.testing.tests.jpa.xml.merge.relationships.EntityMappingsMergeRelationshipsJUnitTestCase

- testCustomerOrdersMapping
- testOrderCustomerMapping
- testItemNameMapping

And model changes have been made through-out the following test model;
org.eclipse.persistence.testing.models.jpa.xml.merge.relationships

Those changes test runtime metadata processing.

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Thursday, March 06, 2008 9:25 AM
Subject: [eclipselink-dev] fix for bug 221658

>A patch file for bug 221658 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=221658
>
> Description: Metadata processing model re-work and clean-up
>
> 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

[Andrei Ilitchev]

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

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

Description: TransformationMapping (doesn't include ReturningPolicy, access
methods, properties - those are items used by other mappings, too; they will
be dealt with separately).

Please send any feedback you may have,

Thanks,

Andrei

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

[Andrei Ilitchev]

Checked in the fix for https://bugs.eclipse.org/bugs/show_bug.cgi?id=211300
Code review by Guy is pending.

Integrated TransformationMapping with eclipselink jpa.

Thanks,

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

[Guy Pelletier]

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

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

 Description: DDL generation of TransformationMapping throws
ClassCastException

Please review and provide any feedback.

 Cheers,
Guy

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

[Guy Pelletier]

Changes have been submitted.

No automated tests added, however DDL generation has been turned on for the
xml extended model which utilizes a transformation mapping with a class
transformer. Doing this exposed the class cast exception. Note, the DDL
generation is to script only and will remain turned on for future testing.

Reviewed by: Andrei Ilitchev

Cheers,
Guy

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

>A patch file for bug 222233 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=222233
>
> Description: DDL generation of TransformationMapping throws
> ClassCastException
>
> Please review and provide any feedback.
>
> Cheers,
> Guy

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

[Guy Pelletier]

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

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

Description: Incorrect schema creation for @BasicCollection with inheritance

Please review and provide any feedback.

Cheers,
Guy

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

[Guy Pelletier]

Changes have been submitted.

No automated tests were added, however DDL generation has been turned on for
the advanced model that exposed the error. The SQL generation was manually
inspected for correctness. NOTE: the DDL generation is to script only and
does not interact with the database.

Reviewed by: Gordon Yorke

Cheers,
Guy

----- Original Message -----
From: "Guy Pelletier" <guy.pelletier@...>
To: "Dev mailing list for Eclipse Persistence Services"
<eclipselink-dev@...>
Sent: Tuesday, March 11, 2008 3:10 PM
Subject: [eclipselink-dev] fix for bug 221648

>A patch file for bug 221648 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=221648
>
> Description: Incorrect schema creation for @BasicCollection with
> inheritance
>
> 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

[Guy Pelletier]

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

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

Description: IdClass with static, transient or abstract causes invalid
primary key class specification exception.

Please review and provide any feedback.

Cheers,
Guy

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

[Guy Pelletier]

Changes have been submitted.

No automated tests were added, however a model change to PartnerLinkPK (from
the fieldaccess.advanced model) exposed
the error/exception. The fix in turn corrects this exception and the
metadata processing completes without errors and
all JPA tests pass.

Reviewed by: Tom Ware

Cheers,
Guy

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

>A patch file for bug 222308 is now available for review.
>
> https://bugs.eclipse.org/bugs/show_bug.cgi?id=222308
>
> Description: IdClass with static, transient or abstract causes invalid
> primary key class specification exception.
>
> 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

[Guy Pelletier]

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