[jbossts-issues] [JBoss JIRA] (JBTM-2203) Remove ReentrantLock from StateManager

[Tom Jenkinson (JIRA)]

    [ https://issues.jboss.org/browse/JBTM-2203?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12983081#comment-12983081 ] 

Tom Jenkinson commented on JBTM-2203:
-------------------------------------

I don't think we can remove the *Mutex() methods from StateManager as these are part of the public API so people that inherit from that class would no longer have access to them (as you saw in the tests you had to modify). Please can you provide a few more details on what the performance win was with removing the Reentrant lock to help evaluate the benefit vs the drawback of the API change?

> Remove ReentrantLock from StateManager
> --------------------------------------
>
>                 Key: JBTM-2203
>                 URL: https://issues.jboss.org/browse/JBTM-2203
>             Project: JBoss Transaction Manager
>          Issue Type: Sub-task
>      Security Level: Public(Everyone can see) 
>          Components: JTA
>    Affects Versions: 5.0.2
>            Reporter: Michael Musgrove
>            Assignee: Michael Musgrove
>            Priority: Minor
>             Fix For: 5.0.3
>
>
> Remove unused lock from StateManager, and add the access methods to LockManager instead



--
This message was sent by Atlassian JIRA
(v6.2.6#6264)