yupeng9 commented on pull request #5631: URL: https://github.com/apache/incubator-pinot/pull/5631#issuecomment-657751504
> For this PR, from Helix dev perspective, it will not impact the correctness but could apply the throttling logic to make the bootstrap longer. But also it brings more stability of the Zookeeper. As long as Pinot is fine with that, this is a good change. Thanks for taking a look and the confirmation > > BTW, Helix team does suggest to use throttling to controller the number of messages. For the number of the threshold, it is up to Pinot team to discuss. > > @yupeng9 Helix provide helix-rest module for supporting lot of REST APIs and running as a separated service. If you would like to use some advanced API, you can try that part. Yes, I have been using `helix-admin-webapp` around it, and it's handy. Per @kishoreg's suggestion it is a good idea to add `helix-rest` to Pinot's cluster management APIs. ---------------------------------------------------------------- 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
