zhangmeng916 commented on pull request #1307: URL: https://github.com/apache/helix/pull/1307#issuecomment-685936146
> Are we going to add HelixAdmin API changes to this PR? @lei-xia I tried to add Helix admin related code in one PR. It would double the PR size, considering all the test cases. I feel it'll make the PR too huge for review purpose. Still think we should separate them. One is for the underlying logic, and the other for user facing API. What do you think? ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
