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

Reply via email to