[ 
https://issues.apache.org/jira/browse/ARTEMIS-1985?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16551556#comment-16551556
 ] 

Benjamin Graf edited comment on ARTEMIS-1985 at 7/21/18 3:00 PM:
-----------------------------------------------------------------

Hi [~clebertsuconic],

I can provide a patch changing default return value for prepare. I'm actually 
thinking about how to write test case because to "really" validate it need a 
transaction manager. Actually no test dependency to JTA transaction manager 
does exist. May I use Geronimo Tx? It's easy to integrate and not a non Apache 
dependency.

(https://github.com/graben/activemq-artemis/tree/ARTEMIS-1985)

What do you think?

Regards

Benjamin


was (Author: graben):
Hi [~clebertsuconic],

I can provide a patch changing default return value for prepare. I'm actually 
thinking about how to write test case because to "really" validate it need a 
transaction manager. Actually no test dependency to JTA transaction manager 
does exist. May I use Geronimo Tx? It's easy to integrate and not a non Apache 
dependency.

What do you think?

Regards

Benjamin

> Produced message via OpenWire does not get committed in XA transaction
> ----------------------------------------------------------------------
>
>                 Key: ARTEMIS-1985
>                 URL: https://issues.apache.org/jira/browse/ARTEMIS-1985
>             Project: ActiveMQ Artemis
>          Issue Type: Bug
>          Components: OpenWire
>    Affects Versions: 2.6.2
>            Reporter: Benjamin Graf
>            Priority: Critical
>             Fix For: 2.7.0, 2.6.3
>
>
> OpenWireConnection actually does return RDONLY as result for prepare. This 
> state causes transaction manager to ignore resource in a 2pc commit. It 
> should be XA_OK as returned by core protocol an ActiveMQ broker.
> To validate you need a XA transaction with min. 2 resources to force 
> transaction manager to not optimize commit to 1pc.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to