accomodate manually initiated rollbacks #1137
Conversation
When methods marked @transactional initiated a rollback without throwing an exception, the Interceptor threw an exception with either "Transaction not active" or "Transaction marked for rollback only".. Fixes google#1136
|
Build on jdk7 failed due to a configuration error, while this PR does not touch any CI-related configuration. Could someone please restart the build and check whether the problem is persistent? |
|
Poking this PR since there recently has been activity on the repository beyond just syncing from internal development. Could I please get some feedback on the state of this PR (and possibly the validity of the issue it's trying to solve?) |
|
We've been eagerly waiting for this to get fixed. As mentioned by the OP, it seems, that the posted commit would work fine, and that the build error was only of temporary nature, completely unrelated to the commit. This bug is quite serious, as it makes guice-persist virtually unusable if the DB connection is not extremely stable (i.e. in many real world situations). |
When methods marked @transactional initiated a rollback without
throwing an exception, the Interceptor threw an exception with
either "Transaction not active" or "Transaction marked for rollback only"..
Fixes #1136