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

Reply via email to