[GitHub] [incubator-pinot] siddharthteotia opened a new pull request #4514: Reduce the number of calls to ZK to fetch the current ideal state in rebalancer

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

siddharthteotia opened a new pull request #4514: Reduce the number of calls to ZK to fetch the current ideal state in rebalancer
URL: https://github.com/apache/incubator-pinot/pull/4514
 
 
   In TableRebalancer, after updating the ideal state (after moving segments), if we successfully persisted this in ZK, we were still going back and fetching the current ideal state from ZK again.
   
   The changes here fetch the ideal state from ZK and re-compute target ideal state only if we failed to persist updated ideal state in ZK earlier due to version error. If there is no version error, we simply start the next iteration (if there are more segments to be moved to get to target) by taking ideal state we just wrote in ZK as the current ideal state.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org