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:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
