Aravind-Suresh commented on code in PR #11183: URL: https://github.com/apache/pinot/pull/11183#discussion_r1281277201
########## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/ServerReloadControllerJobStatusResponse.java: ########## @@ -20,15 +20,25 @@ import java.util.Map; + public class ServerReloadControllerJobStatusResponse { private double _timeElapsedInMinutes; private double _estimatedTimeRemainingInMinutes; private int _totalSegmentCount; + private int _totalCount; Review Comment: So this class was used for reloading all segments API - wanted to share the same response class here and tried to generify this. Progress is successCount / totalCount (didn't want to overload totalSegmentCount for reloadTable as that's very specific to segments and not table as such). The progress for this is how many servers have reloaded their table. -- 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