You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by "Jean-Louis Monteiro (Jira)" <ji...@apache.org> on 2020/11/16 15:00:10 UTC

[jira] [Commented] (TOMEE-2921) [Certification/TCK] Transactions leak into ThreadLocal

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

Jean-Louis Monteiro commented on TOMEE-2921:
--------------------------------------------

Opened a ticket against the TCK

[https://github.com/eclipse-ee4j/jakartaee-tck/issues/573]

 

 

> [Certification/TCK] Transactions leak into ThreadLocal
> ------------------------------------------------------
>
>                 Key: TOMEE-2921
>                 URL: https://issues.apache.org/jira/browse/TOMEE-2921
>             Project: TomEE
>          Issue Type: Bug
>    Affects Versions: 8.0.0-Final, 8.0.1, 8.0.2, 8.0.3, 8.0.4
>            Reporter: Jean-Louis Monteiro
>            Priority: Major
>
> Been trying to look at JTA support and see if TomEE passes TCKs.
> Looks like we are failing on most of the tests in the package 
>  
> com.sun.ts.tests.jta.ee.transactional.*
>  
> After digging into why they were failing I realized, transactions were leaking into the thread context (ThreadLocal) and therefore if the test was randomly picking a test that was badly used before, it would fail on UserTransaction.begin because nested transactions aren't allowed.
>  
> After digging into TomEE and Geronimo Transaction I realized i was probably the TCK test which was not too good.
>  
> Here is where it fails.
> https://github.com/eclipse-ee4j/jakartaee-tck/blob/master/src/com/sun/ts/tests/jta/ee/transactional/Client.java#L420
>  
> I see only begin() but no rollback/commit in some methods which means that some transactions may leak.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)