-
1. Re: envers writes audit entries despite transaction rollback
mdond May 10, 2012 4:48 AM (in response to mdond)Is there any additional information needed or something unclear?
This is a severe problem to us. I tried to describe the problem short but precise and will happily add additional information if anything is unclear.
-
2. Re: envers writes audit entries despite transaction rollback
adamw May 23, 2012 6:31 AM (in response to mdond)Hmm that's really weird ... hard to say anything without a test case running on the latest version
The AuditProcess is associated with a session. So probably opening a new sessino would work. But that's of course only a possible work-around.
Adam
-
3. Re: envers writes audit entries despite transaction rollback
mdond May 23, 2012 6:55 AM (in response to adamw)Hi Adam,
Somehow I expected that answer.
I'm very busy at the moment, but I will try to find some time to reproduce the error in a test case for the 3.6.5-version. This test case can than be used for the latest version.
To me it seems the AuditProcess is still 'subscribed' to a 'doBeforeTransactionCompletion'-event. The removal from auditprocesses in AuditProcessManager doesn't seem to be enough to unsubscribe. An easy patch would be to set some 'removed'-attribute in the AuditProcess and check this attribute in doBeforeTransactionCompletion, but it seems a bit like a hack to me.
I hope, I get some time for this soon...
Maarten
-
4. Re: envers writes audit entries despite transaction rollback
tdziurko Oct 11, 2012 3:44 AM (in response to mdond)Hello Maarten
We have encountered similar issue in our project and with help of Adam we were table to fix it. I've created bug in Hibernate JIRA: https://hibernate.onjira.com/browse/HHH-7682 and pull request with failing test anf fix https://github.com/hibernate/hibernate-orm/pull/393 .
What is strange that this issue is present in Envers 3.6.6 but not in 4.1 version or at least my failing test is passing on 4.1.
-
5. Re: envers writes audit entries despite transaction rollback
mdond Oct 11, 2012 4:04 AM (in response to tdziurko)Hi Tomasz,
This is great news!!! I didn't get any time to create a test as I hoped for in my last post here, so thanks a lot.
I'll check the fix as soon as possible (probably somewhere next week...) and leave a note here about my experience with it.
And some day we probably will migrate to 4.1...
Maarten