[jboss-jira] [JBoss JIRA] (WFLY-13632) If a CMR resource is deployed concurrently then it will overwrite the list of resources that are expected to behave as CMR

[Brian Stansberry (Jira)]

    [ https://issues.redhat.com/browse/WFLY-13632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14202331#comment-14202331 ] 

Brian Stansberry commented on WFLY-13632:
-----------------------------------------

My impression is the basic issue here is that CMResourceService is not dealing with the JTAEnvironmentBean in a thread safe manner. If so it can just synchronize on the bean instance (which should be shared across all CMResourceService instances) or if that's not wanted it can synchronize on a static lock object.

> If a CMR resource is deployed concurrently then it will overwrite the list of resources that are expected to behave as CMR
> --------------------------------------------------------------------------------------------------------------------------
>
>                 Key: WFLY-13632
>                 URL: https://issues.redhat.com/browse/WFLY-13632
>             Project: WildFly
>          Issue Type: Bug
>          Components: Transactions
>            Reporter: Thomas Jenkinson
>            Assignee: Michael Musgrove
>            Priority: Blocker
>
> Inspecting https://github.com/wildfly/wildfly/blob/master/transactions/src/main/java/org/jboss/as/txn/service/CMResourceService.java#L73 we can see that it will overwrite the map in https://github.com/jbosstm/narayana/blob/48818111d12d2e51b52082a00bcb606040c36546/ArjunaJTA/jta/classes/com/arjuna/ats/jta/common/JTAEnvironmentBean.java#L1111 because it first get's a copy of a map (https://github.com/wildfly/wildfly/blob/master/transactions/src/main/java/org/jboss/as/txn/service/CMResourceService.java#L63 -> https://github.com/jbosstm/narayana/blob/48818111d12d2e51b52082a00bcb606040c36546/ArjunaJTA/jta/classes/com/arjuna/ats/jta/common/JTAEnvironmentBean.java#L1089) so the maps won't be equal.
> This will result in WildFly overwriting the list of resources that Narayana will treat as CMR.



--
This message was sent by Atlassian Jira
(v7.13.8#713008)