[GitHub] [incubator-pinot] harishks commented on pull request #6983: Controller api periodic task trigger

[GitBox <gi...@apache.org>]

harishks commented on pull request #6983:
URL: https://github.com/apache/incubator-pinot/pull/6983#issuecomment-855165557


   > What if the task is already running in another controller? Tables are distributed across controllers, and it is assumed in Pinot that when one task runs for a table, it is not running for the same table in another controller.
   
   Thanks for the review. I was basing these changes based on existing periodic tasks, which I believe runs on mutually exclusive set of tables.  Should we maintain states that need to be maintained when the task gets run on a given table on a given controller so that the other controller can skip running these tasks on the same 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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org