This is great news :)

We can finally remove the lame hack in the CmpJpaEngine (or what ever I named that class) that works around this bug. It is the code that tracks the newly created instances so we don't have to ask the JPA code.

-dain

On Jul 10, 2008, at 11:10 AM, David Blevins wrote:


On Jul 10, 2008, at 6:36 AM, Rick McGuire wrote:

I was looking at the JpaTest test cases today that Dain recommended yesterday, and noticed that there were a couple of asserts that were commented out and marked with "FAILS" in the newDeleteNew() method. As an experiment, I updated the build to OpenJPA 1.1.0, and enabled those tests. They now pass on the most recent version of OpenJPA. I suspect we might want to consider upleveling OpenJPA now. This would allow some code in the CMP container that implemented a work- around for this bug to be removed. Some of the other workarounds since as marking the fields that OpenJPA adds as transient might also be removable.

Sounds great to me.


-David


Reply via email to