Jiri Pechanec created RTGOV-329:
-----------------------------------
Summary: Consider refactoring/cleanup of ActivtyServerLogger
Key: RTGOV-329
URL:
https://issues.jboss.org/browse/RTGOV-329
Project: RTGov (Run Time Governance)
Issue Type: Quality Risk
Components: Activity Collector
Affects Versions: 1.0.0.Final
Reporter: Jiri Pechanec
Assignee: Gary Brown
1) I consider the code to be overcomplicated - are we really getting any performance boost
with reusing the ArrayList of events? I found on the web that clear() might be even slower
than new allocation
2) There is unsynced access to fields _sequenceNumber and _failuresSinceLastSuccess
3) I am not sure if it is wise to mix synchronization on Java level - _timer in
BatchedActivityUnitLogger - and Java EE level - method synchornization on @Singleton
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:
http://www.atlassian.com/software/jira