You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@helix.apache.org by GitBox <gi...@apache.org> on 2019/07/26 06:56:41 UTC

[GitHub] [helix] jiajunwang commented on issue #348: Adding the configuration items of the WAGED rebalancer.

jiajunwang commented on issue #348: Adding the configuration items of the WAGED rebalancer.
URL: https://github.com/apache/helix/pull/348#issuecomment-515333872
 
 
   I tried to use more java 8 stream as @i3wangyi suggested. it is convenient.
   But I will not change all of the loops to stream. Some of them are too complicated.
   
   I reviewed the code again myself, and found one issue. Fixed in the new commit.
   Please help to review the logic as well.

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