Eclipse Community Forums: EclipseLink » Problems with @PreUpdate and flush()

Home » Eclipse Projects » EclipseLink » Problems with @PreUpdate and flush()(Problems with @PreUpdate and flush())

Problems with @PreUpdate and flush() [message #498894]

Wed, 18 November 2009 23:37

Eclipse User

Hello all,

I'm trying to add auditing functionality to my entities, using an embedded entity and EntityListeners for updating the audit fields on prePersist and preUpdate.

It works except when an entity is updated and a flush is performed on the EntityManager:
- The @PreUpdate listener is invoked and changes the values
- But DatabaseQueryMechanism.updateObjectForWriteWithChangeSet() does ignore any @PreUpdate changes if it's executed within a EntityManager.flush().

Has anybody had similar problems? Is there a workaround? I'm trying to avoid committing and re-opening a new transaction.

I'm experiencing this problem in EclipseLink v1.1.2, 1.1.3 and 1.2.0.

Thanks
Thomas

Re: Problems with @PreUpdate and flush() [message #498940 is a reply to message #498894]

Thu, 19 November 2009 04:57

Eclipse User

I experienced something similar, which had to do with the fact that these events are called when the object itself is being processed. This can mean that other related objects already where inserted/updated and that changes made to those objects simply are ignored. The events are not fired "en masse" before the actual persisting starts, but during the process. Maybe this is related.

Thomas wrote:
> Hello all,
>
> I'm trying to add auditing functionality to my entities, using an
> embedded entity and EntityListeners for updating the audit fields on
> prePersist and preUpdate.
>
> It works except when an entity is updated and a flush is performed on
> the EntityManager:
> - The @PreUpdate listener is invoked and changes the values
> - But DatabaseQueryMechanism.updateObjectForWriteWithChangeSet() does
> ignore any @PreUpdate changes if it's executed within a
> EntityManager.flush().
>
> Has anybody had similar problems? Is there a workaround? I'm trying to
> avoid committing and re-opening a new transaction.
>
> I'm experiencing this problem in EclipseLink v1.1.2, 1.1.3 and 1.2.0.
>
> Thanks
> Thomas

Re: Problems with @PreUpdate and flush() [message #499127 is a reply to message #498940]

Thu, 19 November 2009 19:36

Eclipse User

Were you able to solve this problem?

I've tracked down the problem where the flush explicitly ignores changes by any events that happened to updated entities (DatabaseQueryMechanism.updateObjectForWriteWithChangeSet(), line 1080 in if uowChangeSet.isChangeSetFromOutsideUOW()) - for some reason it is working when the entities are newly persisted.

After isolating the problem in a small testcase I found the Hibernate does not have this problem.

Cheers
Thomas

Re: Problems with @PreUpdate and flush() [message #499144 is a reply to message #499127]

Fri, 20 November 2009 00:58

Eclipse User

I decided to use an application level event; so before committing I made sure that my application called the logic that normally would be in the @PreUpdate. ATM I could also let my ExtendedEntityManager generate such an event.

Tom

Thomas wrote:
> Were you able to solve this problem?
>
> I've tracked down the problem where the flush explicitly ignores changes
> by any events that happened to updated entities
> (DatabaseQueryMechanism.updateObjectForWriteWithChangeSet(), line 1080
> in if uowChangeSet.isChangeSetFromOutsideUOW()) - for some reason it is
> working when the entities are newly persisted.
>
> After isolating the problem in a small testcase I found the Hibernate
> does not have this problem.
>
> Cheers
> Thomas

Re: Problems with @PreUpdate and flush() [message #499690 is a reply to message #498894]

Mon, 23 November 2009 13:05

Eclipse User

In order to raise the preUpdate events, the object's changes must be computed, if you then change the object again in this event, then new changes will not be picked up (as this would require us re-computing the changes again, which would be twice as much work, but might be a useful option to have).

You can use the DescriptorEvent preUpdate instead of the JPA event, this will give you access to the DescriptorEvent object, which has an API, updateAttributeWithObject(), which will update the object, row and change set.

Re: Problems with @PreUpdate and flush() [message #499756 is a reply to message #499690]

Mon, 23 November 2009 19:32

Eclipse User

James,
thanks for that. I was trying to avoid any provider specific API and isn't JPA's preUpdate event intended to give an EntityListener the chance to modify the updated objects? Mind you, that it all works fine if you don't do a flush() and end the transaction normally. So I think it's a bug with EclipseLink's flush.

Cheers
Thomas

Re: Problems with @PreUpdate and flush() [message #663762 is a reply to message #499756]

Wed, 06 April 2011 08:47

Eclipse User

Thomas wrote on Tue, 24 November 2009 07:32

Is still happening with the latest EclipseLink ?

Will the following code work as intended ?

@PreUpdate
@PrePersist
public void updateTimeStamps() {
    lastModified = new Date();
    if (dateCreated==null) {
      dateCreated = new Date();
    }
}

Re: Problems with @PreUpdate and flush() [message #663966 is a reply to message #663762]

Wed, 06 April 2011 20:46

Eclipse User

Hi Hendy,

Haven't tried it myself anymore, but had raised a bug for Eclipselink: https://bugs.eclipse.org/bugs/show_bug.cgi?id=295665
According to that ticket it's been fixed from 1.2.1 onwards.

Cheers
Thomas

Re: Problems with @PreUpdate and flush() [message #664347 is a reply to message #663966]

Fri, 08 April 2011 09:28

Eclipse User

Thomas wrote on Thu, 07 April 2011 07:46

Indeed, I've tried it with EclipseLink on GlassFish 3.1 and code in @PreUpdate can modify the entity just fine Smile

Previous Topic:	JAXB webservice exception
Next Topic:	UnitOfWork.beginEarlyTransaction not having expected effect

Goto Forum:

-=] Back to Top [=-

Current Time: Sat May 03 07:55:02 EDT 2025

.:: Contact :: Home ::.

Breadcrumbs

Sign up to our Newsletter