[GitHub] [helix] jiajunwang commented on issue #342: Kickoff for the Weight Aware Globally Even Distribute Rebalancer

2019-08-02 Thread GitBox
jiajunwang commented on issue #342: Kickoff for the Weight Aware Globally Even Distribute Rebalancer URL: https://github.com/apache/helix/issues/342#issuecomment-517893119 The foundation of the new rebalancer has been built after the interface/cluster model checked in. I'm closing this

[GitHub] [helix] jiajunwang closed issue #342: Kickoff for the Weight Aware Globally Even Distribute Rebalancer

2019-08-02 Thread GitBox
jiajunwang closed issue #342: Kickoff for the Weight Aware Globally Even Distribute Rebalancer URL: https://github.com/apache/helix/issues/342 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [helix] jiajunwang merged pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang merged pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [helix] jiajunwang commented on issue #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on issue #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#issuecomment-517892944 This PR is ready to be merged, approved by @narendly. I will do the merge.

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310338335 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336821 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336903 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336872 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336640 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336694 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/AssignableNode.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336640 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310336585 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/AssignableNode.java

[GitHub] [helix] jiajunwang opened a new issue #372: Implement the Cluster Model data provider for the WAGED rebalancer

2019-08-02 Thread GitBox
jiajunwang opened a new issue #372: Implement the Cluster Model data provider for the WAGED rebalancer URL: https://github.com/apache/helix/issues/372 Please find the design doc here:

[GitHub] [helix] jiajunwang commented on issue #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on issue #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#issuecomment-517854427 Thanks for your hard work of reviewing. I know it is a huge PR. Mainly because it is new classes that are built from scratch, even

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310284240 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310279995 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310280122 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310278490 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310277680 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310277478 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310277080 ## File path:

Missing 0.9.0.1 Helix release in maven repository

2019-08-02 Thread Xue Junkai
Hi Folks, We are generating an Apache Helix release 0.9.0.1. Everything looks good after we click the release in Repository Manager. But we did not find the 0.9.0.1 release in maven repository until now. Could you please help us figure it out why it is not showing up? It was released yesterday

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310276844 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310249111 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310248906 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310246676 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310245795 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310244594 ## File path:

[GitHub] [helix] jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
jiajunwang commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310244369 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310237287 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310238349 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239674 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310236475 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310237911 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239447 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239990 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239874 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239254 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310239855 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310236475 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310234688 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310234688 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310233448 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310233448 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310232277 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310232277 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310230832 ## File path:

[GitHub] [helix] i3wangyi commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
i3wangyi commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310226396 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r309939372 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310213495 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310218704 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310212807 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310170701 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310223885 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r309921368 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterContext.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310213728 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310214870 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r309939522 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310217077 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310216570 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310216339 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310214018 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310162365 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310215326 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r309939616 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on issue #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on issue #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#issuecomment-517781121 This is a big PR! I think it would be a good idea to keep the size of PRs smaller if possible.

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310221038 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310213569 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310218571 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310216686 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310215210 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310213021 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310215940 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310161710 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/ClusterModel.java

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310218199 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation

2019-08-02 Thread GitBox
narendly commented on a change in pull request #362: The WAGED rebalancer cluster model implementation URL: https://github.com/apache/helix/pull/362#discussion_r310213353 ## File path:

[GitHub] [helix] alirezazamani closed issue #370: Remove unnecessary touch logic when updating ResourceConfig

2019-08-02 Thread GitBox
alirezazamani closed issue #370: Remove unnecessary touch logic when updating ResourceConfig URL: https://github.com/apache/helix/issues/370 This is an automated message from the Apache Git Service. To respond to the

helix - Build # 1638 - Still Failing

2019-08-02 Thread Apache Jenkins Server
The Apache Jenkins build system has built helix (build #1638) Status: Still Failing Check console output at https://builds.apache.org/job/helix/1638/ to view the results.

[GitHub] [helix] dasahcc merged pull request #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines

2019-08-02 Thread GitBox
dasahcc merged pull request #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines URL: https://github.com/apache/helix/pull/371 This is an automated message from the Apache Git Service. To

[GitHub] [helix] alirezazamani commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines

2019-08-02 Thread GitBox
alirezazamani commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines URL: https://github.com/apache/helix/pull/371#issuecomment-517754322 This PR is ready to be merged, approved by @jiajunwang and @narendly.

[GitHub] [helix] alirezazamani commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines

2019-08-02 Thread GitBox
alirezazamani commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines URL: https://github.com/apache/helix/pull/371#issuecomment-517754114 These changes have been covered by tests. I also check it manually and the pipeline is triggered.

[GitHub] [helix] narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight …

2019-08-02 Thread GitBox
narendly commented on a change in pull request #369: Add the workaround fix for assigning partitions when instance weight … URL: https://github.com/apache/helix/pull/369#discussion_r310154419 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310153372 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310153372 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310150104 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config

2019-08-02 Thread GitBox
narendly commented on a change in pull request #333: Fix issue when client only sets ANY at cluster level throttle config URL: https://github.com/apache/helix/pull/333#discussion_r310150588 ## File path:

[GitHub] [helix] narendly commented on a change in pull request #359: Dynamically change the processor thread name when consuming event

2019-08-02 Thread GitBox
narendly commented on a change in pull request #359: Dynamically change the processor thread name when consuming event URL: https://github.com/apache/helix/pull/359#discussion_r310147891 ## File path: helix-core/src/main/java/org/apache/helix/controller/GenericHelixController.java

[GitHub] [helix] narendly commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines

2019-08-02 Thread GitBox
narendly commented on issue #371: Remove unnecessary touch logic issues that forces triggering Helix pipelines URL: https://github.com/apache/helix/pull/371#issuecomment-517712533 You could do https://github.com/apache/helix/wiki/Pull-Request-Check-In-Steps to get this checked in. As