New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HHH-13433 HHH-13737 (5.3) #3100
Conversation
8d66a78
to
34fcf8f
Compare
…ondition if there is an active JTA transaction, otherwise ORM should throw throw convert( e, lockOptions )
34fcf8f
to
80cb8d2
Compare
// Just log the exception and return null. | ||
if ( log.isDebugEnabled() ) { | ||
log.debug( "JDBCException was thrown for a transaction marked for rollback; " + | ||
"this is probably due to IronJacamar failing fast after " + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be okay to generalize the logged message to not mention IronJacamar, in case IronJacamar is not in use, perhaps something like:
log.debug( "JDBCException was thrown for a transaction marked for rollback; " +
"this is probably due to an operation failing fast due to " +
"transaction marked for rollback.", e );
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@scottmarlow , yes, good idea; I'll do that.
I see there's a checkstyle failure, so I'll fix that as well.
@scottmarlow , I just pushed a commit with the change you requested, and a fix for the checkstyle failure. |
Looks good! :) |
@scottmarlow , thanks Scott! I'll leave this one open until it's approved for 5.3 branch. I'll port the HHH-13737 to master. |
https://hibernate.atlassian.net/browse/HHH-13433
https://hibernate.atlassian.net/browse/HHH-13737
Once approved, the same test case and logging will get pushed to master.