dujian0068 commented on code in PR #16266:
URL: https://github.com/apache/kafka/pull/16266#discussion_r1635728504
##########
trogdor/src/main/java/org/apache/kafka/trogdor/workload/SustainedConnectionWorker.java:
##########
@@ -60,7 +60,7 @@
public class SustainedConnectionWorker implements TaskWorker {
private static final Logger log =
LoggerFactory.getLogger(SustainedConnectionWorker.class);
- private static final SystemTime SYSTEM_TIME = new SystemTime();
+ private static final Time SYSTEM_TIME = Time.SYSTEM;
Review Comment:
You are right,
After updating `SystemTime` to singleton mode, many class attributes `time`
can be modified inline, but I am not sure whether to modify it because it is a
large-scale change to modify many files.
--
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: jira-unsubscr...@kafka.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
