jiajunwang commented on issue #639: Refine the WAGED rebalancer to minimize the partial rebalance workload. URL: https://github.com/apache/helix/pull/639#issuecomment-560686297 > Could adding the "algorithm" field in the constructor have been in a separate PR as well? There are a lot of moving parts here and very few people have the right context. It would have been better if that part, however small or trivial it is, could have been separated into another PR as well (which would have been an easy approval). We are not rush to have the approval soon. But creating too much PRs may confuse myself because of too many threads. I'm trying to split the original PR by files. If the same file appears in more than one sub-PRs, there is a high chance I will need to resolve conflict or miss something.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
