Github user kishoreg commented on the pull request:
https://github.com/apache/helix/pull/41#issuecomment-209564520
Thanks, will apply the change today
On Wed, Apr 13, 2016 at 10:46 AM, Lei Xia wrote:
> Rebased to the head.
>
> â
Github user kishoreg commented on the pull request:
https://github.com/apache/helix/pull/41#issuecomment-182482218
Will review this today
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user kishoreg commented on the pull request:
https://github.com/apache/helix/pull/41#issuecomment-182487167
Looks good, please try to split the changes into smaller RB's. Its very
hard to review such a big RB. Can you address the RB comments. I will merge it
---
If your
Github user kishoreg commented on a diff in the pull request:
https://github.com/apache/helix/pull/41#discussion_r52488945
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java.rej
---
@@ -0,0 +1,18 @@
+diff
Github user kishoreg commented on a diff in the pull request:
https://github.com/apache/helix/pull/41#discussion_r52489589
--- Diff: helix-core/src/main/java/org/apache/helix/task/JobRebalancer.java
---
@@ -206,9 +205,23 @@ private ResourceAssignment
computeResourceMapping(String
Github user lei-xia commented on the pull request:
https://github.com/apache/helix/pull/41#issuecomment-182658641
Thanks for reviewing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user lei-xia commented on a diff in the pull request:
https://github.com/apache/helix/pull/41#discussion_r52551298
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java.rej
---
@@ -0,0 +1,18 @@
+diff
Github user lei-xia commented on the pull request:
https://github.com/apache/helix/pull/41#issuecomment-182151763
Ping for reviewing!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
GitHub user lei-xia opened a pull request:
https://github.com/apache/helix/pull/41
A few more task framework improvement
This pull request includes four diffs (with each described as below):
1. [HELIX-622] Add new resource configuration option to allow resource to
disable