somandal commented on code in PR #15510:
URL: https://github.com/apache/pinot/pull/15510#discussion_r2047774864
##########
pinot-controller/src/main/java/org/apache/pinot/controller/helix/core/rebalance/ZkBasedTableRebalanceObserver.java:
##########
@@ -100,6 +110,12 @@ public void onTrigger(Trigger trigger, Map<String,
Map<String, String>> currentS
}
break;
case EXTERNAL_VIEW_TO_IDEAL_STATE_CONVERGENCE_TRIGGER:
+ // Update the previous stats with the current values in case a
rollback is needed due to IdealState version
+ // change
+ _previousStepStats = new
TableRebalanceProgressStats.RebalanceProgressStats(
Review Comment:
yes that is correct
--
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
