KKcorps commented on code in PR #14811: URL: https://github.com/apache/pinot/pull/14811#discussion_r1920077754
########## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotRealtimeTableResource.java: ########## @@ -170,19 +169,28 @@ public Map<String, String> forceCommit( @ApiParam(value = "Comma separated list of partition group IDs to be committed") @QueryParam("partitions") String partitionGroupIds, @ApiParam(value = "Comma separated list of consuming segments to be committed") @QueryParam("segments") - String consumingSegments, @Context HttpHeaders headers) { + String consumingSegments, + @ApiParam(value = "Max number of consuming segments to commit at once") @QueryParam("batchSize") Review Comment: mention that default is all consuming segments at once in description -- 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