Re: [hibernate-dev] afterTransactionCompletion(status) in JtaTransaction

2012-07-18 Thread Steve Ebersole
I don't see an issue with changing that signature. On 07/18/2012 10:27 AM, Sanne Grinovero wrote: > Hi Steve, > thanks for the clarification. > > Wouldn't it make sense then to add the "int status" as an argument to > afterAfterTransaction as well? As it seems some code looks for this by > invokin

Re: [hibernate-dev] afterTransactionCompletion(status) in JtaTransaction

2012-07-18 Thread Sanne Grinovero
Hi Steve, thanks for the clarification. Wouldn't it make sense then to add the "int status" as an argument to afterAfterTransaction as well? As it seems some code looks for this by invoking tx.getStatus() in implementations of this method, but that seems unsafe as I've found cases in which this ge

Re: [hibernate-dev] afterTransactionCompletion(status) in JtaTransaction

2012-07-18 Thread Steve Ebersole
afterAfterTransaction is simply a part of the afterTransaction process. afterTransaction is split into a number of steps, afterAfterTransaction simply being the last step in that process. On 07/18/2012 05:15 AM, Sanne Grinovero wrote: > Hi all, > I'm puzzled about the implementation we have in >

[hibernate-dev] afterTransactionCompletion(status) in JtaTransaction

2012-07-18 Thread Sanne Grinovero
Hi all, I'm puzzled about the implementation we have in org.hibernate.engine.transaction.internal.jta.JtaTransaction regarding after-transaction completion handling. Looking at the two methods: afterAfterCompletion(); <-- note the "After - After" prefix afterTransactionCompletion(int status);