jackjlli commented on pull request #5724: URL: https://github.com/apache/incubator-pinot/pull/5724#issuecomment-662066684
> If I understand this right, we are disabling quota check on brokers dynamically. Assuming such dynamic enable/disable is a requirement, why not provide a (rest) API to do it? Why use helix messages? The reason why I put the API to pinot controller and send a Helix message to broker is that we've already been using this way to update query quotas to brokers (when table config is updated). Plus, all the logic of cluster management should all be in one place, which is in Pinot controller. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
