[jbossts-issues] [JBoss JIRA] (JBTM-3162) Remove superfluous double check at validTransaction method
Thomas Jenkinson (Jira)
issues at jboss.org
Wed Jul 24 11:22:00 EDT 2019
[ https://issues.jboss.org/browse/JBTM-3162?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13762954#comment-13762954 ]
Thomas Jenkinson commented on JBTM-3162:
----------------------------------------
Thanks, please can you raise a PR for it over here: https://github.com/jbosstm/narayana/pulls
> Remove superfluous double check at validTransaction method
> ----------------------------------------------------------
>
> Key: JBTM-3162
> URL: https://issues.jboss.org/browse/JBTM-3162
> Project: JBoss Transaction Manager
> Issue Type: Enhancement
> Components: Transactional Driver
> Affects Versions: 5.9.5.Final
> Reporter: Ariel Carrera
> Priority: Trivial
>
> There is a superfluous check at BaseTransactionalDriverXAConnection.validTransaction method.
> {code:java}
> public final boolean validTransaction(javax.transaction.Transaction tx) {
> boolean valid = true;
> if (_theTransaction != null) {
> valid = _theTransaction.equals(tx);
> * if (!valid) {
> valid = _theTransaction.equals(tx);
> } *
> }
> return valid;
> }
> {code}
> link:
> [github|https://github.com/jbosstm/narayana/blob/9f42d45b664483c9f0e17cb20122c1d5436ba856/ArjunaJTA/jdbc/classes/com/arjuna/ats/internal/jdbc/BaseTransactionalDriverXAConnection.java#L81]
--
This message was sent by Atlassian Jira
(v7.12.1#712002)
More information about the jbossts-issues
mailing list