jungm commented on PR #1996:
URL: https://github.com/apache/tomee/pull/1996#issuecomment-3130863438

   quick side note, the commit that caused all this is 
https://github.com/apache/tomee/pull/1458/commits/fb6340de805c5454d61d9bbadf25f5fcad730ee3
   
   I'm not exactly sure anymore why i chose to do it this way, but removing it 
now makes the concurrency TCK fail:
   ```
   [ERROR] Failures: 
   [ERROR]   
TransactionTests>Arquillian.run:137->testCancelTransactionWithManagedScheduledExecutorService:89->TestClient.runTest:70->TestClient.assertSuccessfulURLResponse:143->ArquillianTests.assertTrue:108
 Output did not contain successful message: SUCCESS expected [true] but found 
[false]
   [ERROR]   
TransactionTests>Arquillian.run:137->testRollbackTransactionWithManagedScheduledExecutorService:72->TestClient.runTest:70->TestClient.assertSuccessfulURLResponse:143->ArquillianTests.assertTrue:108
 Output did not contain successful message: SUCCESS expected [true] but found 
[false]
   [INFO] 
   [ERROR] Tests run: 147, Failures: 2, Errors: 0, Skipped: 1
   
   ```
    Might need to look at this again and figure out another way to pass these 
tests


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to