chiacyu commented on code in PR #8096:
URL: https://github.com/apache/ozone/pull/8096#discussion_r1997535360


##########
hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/fsck/ContainerHealthTask.java:
##########
@@ -146,8 +146,8 @@ protected void runTask() throws Exception {
       initializeUnhealthyContainerStateStatsMap(
           unhealthyContainerStateStatsMap);
       long start = Time.monotonicNow();
-      long currentTime = System.currentTimeMillis();
-      long existingCount = processExistingDBRecords(currentTime,
+      long currentTime = Time.monotonicNow();
+      long existingCount = processExistingDBRecords(Time.monotonicNow(),

Review Comment:
   Hi, @peterxcli 
   Thanks for the review and comments. In the following line the `start` would 
be subtracted to estimate the duration, please take a look and tell me what do 
you think. Thanks!
   ```java
         LOG.debug("Container Health task thread took {} milliseconds to" +
                 " process {} existing database records.",
             Time.monotonicNow() - start, existingCount);
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to