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