[ http://issues.apache.org/jira/browse/GERONIMO-550?page=comments#action_58192 ] David Jencks commented on GERONIMO-550: ---------------------------------------
I've applied the changes that remove the timer/interrupt and just check the time at commit, and also I reset the thread timeout value after it has been used. I added the test. I did not convert the internal time unit to seconds. I kept the use of the ExtendedTransactionManager interface to start a transaction while specifying a timeout to avoid unnecessary use of a thread local. I'd still like to investigate whether we can write a thread-safe timer/interrupt timeout mechanism, but perhaps it should be an alternative to this one. Revision 128441 > Functioning of Transaction Time out was not properly handled > ------------------------------------------------------------ > > Key: GERONIMO-550 > URL: http://issues.apache.org/jira/browse/GERONIMO-550 > Project: Apache Geronimo > Type: Bug > Components: transaction manager > Versions: 1.0-M4 > Environment: All platforms and All JDK Version > Reporter: Sandip Ghayal > Assignee: David Jencks > Attachments: TransactionTimer.java, patches.out > > Currently operation for timeout for transaction is not thread safe -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - If you want more information on JIRA, or have a bug to report see: http://www.atlassian.com/software/jira
