Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/91#discussion_r115862430
--- Diff:
helix-core/src/main/java/org/apache/helix/messaging/handling/HelixTaskExecutor.java
---
@@ -210,6 +213,18 @@ private void
[
https://issues.apache.org/jira/browse/HELIX-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005448#comment-16005448
]
ASF GitHub Bot commented on HELIX-655:
--
Github user kongweihan commented on a diff in the pull
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115829596
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/BestPossibleStateCalcStage.java
---
@@ -204,4 +236,28 @@ private MappingCalculator
[
https://issues.apache.org/jira/browse/HELIX-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005449#comment-16005449
]
ASF GitHub Bot commented on HELIX-655:
--
Github user kongweihan commented on a diff in the pull
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115848245
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/CurrentStateOutput.java
---
@@ -260,13 +260,72 @@ public Message
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115849569
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/CurrentStateOutput.java
---
@@ -260,13 +260,72 @@ public Message
[
https://issues.apache.org/jira/browse/HELIX-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005451#comment-16005451
]
ASF GitHub Bot commented on HELIX-655:
--
Github user kongweihan commented on a diff in the pull
[
https://issues.apache.org/jira/browse/HELIX-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005446#comment-16005446
]
ASF GitHub Bot commented on HELIX-655:
--
Github user kongweihan commented on a diff in the pull
[
https://issues.apache.org/jira/browse/HELIX-655?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16005445#comment-16005445
]
ASF GitHub Bot commented on HELIX-655:
--
Github user kongweihan commented on a diff in the pull
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115853776
--- Diff: helix-core/src/main/java/org/apache/helix/task/JobRebalancer.java
---
@@ -704,4 +712,30 @@ private PartitionAssignment(String instance, String
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115847749
--- Diff:
helix-core/src/main/java/org/apache/helix/controller/stages/CurrentStateOutput.java
---
@@ -260,13 +260,72 @@ public Message
Github user kongweihan commented on a diff in the pull request:
https://github.com/apache/helix/pull/89#discussion_r115852400
--- Diff: helix-core/src/main/java/org/apache/helix/task/JobRebalancer.java
---
@@ -57,6 +57,7 @@
new FixedTargetTaskAssignmentCalculator();
Yes. I have the PR for this : https://github.com/apache/helix/pull/91
Best,
Junkai
On Wed, May 10, 2017 at 12:08 PM, kishore g wrote:
> Yes. Do you have a PR for that?. I can review it
>
> On Wed, May 10, 2017 at 11:19 AM, Xue Junkai wrote:
>
> >
GitHub user dasahcc opened a pull request:
https://github.com/apache/helix/pull/91
[Helix-656] Support customize batch state transition thread pool
To better support batch message handling, we shall make batch state
transition thread pool configurable. This config can be put in
Yes. Do you have a PR for that?. I can review it
On Wed, May 10, 2017 at 11:19 AM, Xue Junkai wrote:
> Sure! Please let me know if this change works or not. BTW will customized
> batch message threadpool be involved in this release?
>
> Best,
>
> Junkai
>
> On Tue, May 9,
Github user asfgit closed the pull request at:
https://github.com/apache/helix/pull/90
---
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 feature
enabled and wishes so, or if the feature is
Sure! Please let me know if this change works or not. BTW will customized
batch message threadpool be involved in this release?
Best,
Junkai
On Tue, May 9, 2017 at 7:28 PM, kishore g wrote:
> I would like to have that fix included for Pinot. I will test it the patch.
>
>
17 matches
Mail list logo