J-HowHuang commented on code in PR #15518:
URL: https://github.com/apache/pinot/pull/15518#discussion_r2060641042


##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/ZkBasedTableRebalanceObserver.java:
##########
@@ -234,6 +238,35 @@ public int getNumUpdatesToZk() {
     return _numUpdatesToZk;
   }
 
+  /**
+   * Emits the rebalance progress in percent to the metrics. Uses the 
percentage of remaining segments to be added as
+   * the indicator of the overall progress.
+   * Notice that for some jobs, the metrics may not be exactly accurate and 
would not be 100% when the job is done.
+   * (e.g. when `lowDiskMode=false`, the job finishes without waiting for 
`totalRemainingSegmentsToBeDeleted` become 0)

Review Comment:
   done



-- 
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: commits-unsubscr...@pinot.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to