vlsi commented on code in PR #6353:
URL: https://github.com/apache/jmeter/pull/6353#discussion_r1775517850
##########
src/core/src/main/java/org/apache/jmeter/threads/AbstractThreadGroup.java:
##########
@@ -86,6 +87,8 @@ public abstract class AbstractThreadGroup extends
AbstractTestElement
private final AtomicInteger numberOfThreads = new AtomicInteger(0); //
Number of active threads in this group
+ private final AtomicLong startTime = new AtomicLong(0);
Review Comment:
I guess the sequence is:
1) Thread group initializes
2) Start time is set
3) Thread group starts
4) Timers operate
The timers should not modify "start time", so there's no need to synchronize
the access.
--
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...@jmeter.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
