[
https://issues.apache.org/jira/browse/ARTEMIS-4652?focusedWorklogId=906236&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-906236
]
ASF GitHub Bot logged work on ARTEMIS-4652:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 21/Feb/24 12:45
Start Date: 21/Feb/24 12:45
Worklog Time Spent: 10m
Work Description: clebertsuconic commented on code in PR #4828:
URL: https://github.com/apache/activemq-artemis/pull/4828#discussion_r1497478340
##########
artemis-core-client/src/main/java/org/apache/activemq/artemis/core/client/impl/ClientSessionImpl.java:
##########
@@ -1901,18 +1901,10 @@ public void rollback(final Xid xid) throws XAException {
workDone = false;
} catch (XAException xae) {
throw xae;
- } catch (ActiveMQException e) {
- if (e.getType() == ActiveMQExceptionType.UNBLOCKED || e.getType() ==
ActiveMQExceptionType.CONNECTION_TIMEDOUT || e.getType() ==
ActiveMQExceptionType.SHUTDOWN_ERROR) {
- // Unblocked on failover
- throw new XAException(XAException.XA_RETRY);
- }
-
- // This should never occur
- XAException xaException = new XAException(XAException.XAER_RMFAIL);
- xaException.initCause(e);
- throw xaException;
} catch (Throwable t) {
- // This could occur if the TM interrupts the thread
+ if (logger.isTraceEnabled()) {
+ logger.trace("Rollback failed:: {}", convert(xid), t);
+ }
XAException xaException = new XAException(XAException.XAER_RMFAIL);
Review Comment:
@ehsavoie should we merge this or wait your tests then?
Issue Time Tracking
-------------------
Worklog Id: (was: 906236)
Time Spent: 40m (was: 0.5h)
> Rollback of XAResource implementation should never return XA_RETRY
> ------------------------------------------------------------------
>
> Key: ARTEMIS-4652
> URL: https://issues.apache.org/jira/browse/ARTEMIS-4652
> Project: ActiveMQ Artemis
> Issue Type: Bug
> Reporter: Justin Bertram
> Assignee: Justin Bertram
> Priority: Major
> Time Spent: 40m
> Remaining Estimate: 0h
>
> According to [the XA
> spec|https://pubs.opengroup.org/onlinepubs/009680699/toc.pdf] {{XA_RETRY}} is
> not a valid response from {{xa_rollback}}. We should return {{XAER_RMFAIL}}
> instead.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
