mcvsubbu commented on pull request #6020: URL: https://github.com/apache/incubator-pinot/pull/6020#issuecomment-703717382
> @mcvsubbu The issue is the changes are needed for new /Brokers api. > The plan is to call the API to fetch the brokers for a tenant from JDBC driver. The current implementation requires too many calls to zookeeper. > I was not aware of the watchers concern though. @kishoreg @fx19880617 What should we do about this? Why is calls to zookeeper bad? How many calls is it? Zookeepers can handle a lot more calls to it than pinot controllers can. ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
