vvivekiyer commented on code in PR #12459: URL: https://github.com/apache/pinot/pull/12459#discussion_r1540440738
########## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotTaskRestletResource.java: ########## @@ -615,12 +615,15 @@ public Map<String, String> scheduleTasks(@ApiParam(value = "Task type") @QueryPa tableName = DatabaseUtils.translateTableName(tableName, headers); Review Comment: Would it also make sense to extend this API to take in a `minionInstanceTag`? I understand that `tasks/execute` API already provides that capability. We plan to use this feature as follows: * Say all minion periodic tasks are usually executed in the minion_untagged hosts. * If our minion purge tasks are running behind for a table, we plan to deploy new set of hosts with a special tag e.g. `minion_emergency`, and use the schedule/execute API for one-off scheduling of these jobs on these hosts. -- 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