[GitHub] [incubator-pinot] dasahcc edited a comment on pull request #5631: Add the message constraint on all instances in Helix

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

dasahcc edited a comment on pull request #5631:
URL: https://github.com/apache/incubator-pinot/pull/5631#issuecomment-657737205


   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. 
   
   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.
    


----------------------------------------------------------------
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