klsince commented on code in PR #11740:
URL: https://github.com/apache/pinot/pull/11740#discussion_r1347845423


##########
pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTableRestletResource.java:
##########
@@ -615,11 +615,14 @@ public RebalanceResult rebalance(
   @QueryParam("externalViewCheckIntervalInMs") long 
externalViewCheckIntervalInMs,
       @ApiParam(value = "How long to wait till external view converges with 
ideal states") @DefaultValue("3600000")
       @QueryParam("externalViewStabilizationTimeoutInMs") long 
externalViewStabilizationTimeoutInMs,
+      @ApiParam(value = "How often to make a status update (i.e. heartbeat)") 
@DefaultValue("300000")

Review Comment:
   good catch, will add. I thought the rebalanceStatus API would simply return 
the whole jobMetadata which will include the retry info.



-- 
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