[ 
https://issues.apache.org/jira/browse/GERONIMO-3520?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12533925
 ] 

Matt Hogstrom commented on GERONIMO-3520:
-----------------------------------------

Viet, for the txmanager patch I think there may be an issue with the use of the 
static long.  Since the access is not synchronized there exists a strong 
possibility of lost updates in a heavily concurrent system which would lead to 
inaccurate values.  volatile is one way  to address this problem but I don't 
think it is perfect on all VMs so I think the better solution would be the use 
of java.util.concurrent.atomic.AtomicLong .getAndIncrement() which should do 
the trick.  Not sure of other folks have suggestions.

> missing jsr-77 stats for JTA
> ----------------------------
>
>                 Key: GERONIMO-3520
>                 URL: https://issues.apache.org/jira/browse/GERONIMO-3520
>             Project: Geronimo
>          Issue Type: Improvement
>      Security Level: public(Regular issues) 
>          Components: monitoring, transaction manager
>    Affects Versions: 2.0.2, 2.1
>         Environment: linux
>            Reporter: Viet Hung Nguyen
>         Attachments: geronimo-3520.patch, 
> jms-mdb-sample-ear-2.0-SNAPSHOT.ear, txmanagerStats.patch
>
>
> we need to surface transaction statistics defined by the JSR 77 spec.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to