[jbossts-issues] [JBoss JIRA] (JBTM-2673) ThreadUtil getThreadId(Thread) ignores argument

[Tom Jenkinson (JIRA)]

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

Tom Jenkinson commented on JBTM-2673:
-------------------------------------

Analysis shows that our usage of this internally (that doesn't use Thread.currentThread() as the argument) is:
1. BasicAction::addChildThread(Thread) which is a public method but is unused by ourselves
2. BasicAction::removeAllChildThreads - this is called by the reaper but because typical usage is via JTA which requires the app thread to disassociate we have not had reports on this

> ThreadUtil getThreadId(Thread) ignores argument
> -----------------------------------------------
>
>                 Key: JBTM-2673
>                 URL: https://issues.jboss.org/browse/JBTM-2673
>             Project: JBoss Transaction Manager
>          Issue Type: Bug
>            Reporter: Tom Jenkinson
>            Assignee: Tom Jenkinson
>             Fix For: 5.next
>
>
> During investigation of https://developer.jboss.org/thread/269784?start=15&tstart=0 it was observed that the ThreadUtil implementation does not use the Thread argument and simply uses a ThreadLocal:
> https://github.com/jbosstm/narayana/blob/master/ArjunaCore/arjuna/classes/com/arjuna/ats/arjuna/utils/ThreadUtil.java#L51



--
This message was sent by Atlassian JIRA
(v6.4.11#64026)