[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314883260 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/AssignableNode.java

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314872913 ## File path: helix-core/src/test/java/org/apache/helix/controller/rebalancer/waged/model/TestAssignableNode.java

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314872913 ## File path: helix-core/src/test/java/org/apache/helix/controller/rebalancer/waged/model/TestAssignableNode.java

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314872555 ## File path: helix-core/src/main/java/org/apache/helix/model/ClusterConfig.java ## @@ -700,6 +703,48 @@

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314871440 ## File path: helix-core/src/main/java/org/apache/helix/model/ClusterConfig.java ## @@ -700,6 +703,48 @@

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314870911 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/AssignableNode.java

[GitHub] [helix] jiajunwang commented on a change in pull request #413: Add cluster level default instance config.

2019-08-16 Thread GitBox
jiajunwang commented on a change in pull request #413: Add cluster level default instance config. URL: https://github.com/apache/helix/pull/413#discussion_r314870372 ## File path: helix-core/src/main/java/org/apache/helix/controller/rebalancer/waged/model/AssignableNode.java