narendly opened a new pull request #301: Add markdown for quota-based scheduling
URL: https://github.com/apache/helix/pull/301
This is an automated message from the Apache Git Service.
To respond to the message, please log on
narendly opened a new pull request #302: HELIX: Fix null tgtSessionId message
bug
URL: https://github.com/apache/helix/pull/302
It has been reported that the Controller sometimes sends a message with a
null tgtSessionId, which was causing an NPE on the Participant.
Changelist:
1.
asfgit merged pull request #302: HELIX: Fix null tgtSessionId message bug
URL: https://github.com/apache/helix/pull/302
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
dasahcc commented on issue #301: Add markdown for quota-based scheduling
URL: https://github.com/apache/helix/pull/301#issuecomment-463028870
You may need to move this to 0.8.3 folder, since this feature is released
out from 0.8.3.
--
narendly commented on issue #301: Add markdown for quota-based scheduling
URL: https://github.com/apache/helix/pull/301#issuecomment-463036989
Thanks for noticing. I have moved it to 0.8.3 folder and removed from src.
This is
dasahcc commented on a change in pull request #301: Add markdown for
quota-based scheduling
URL: https://github.com/apache/helix/pull/301#discussion_r258744277
##
File path: website/0.8.3/src/site/markdown/quota_scheduling.md
##
@@ -0,0 +1,181 @@
+
+
+# Quota-based Task Sc
dasahcc commented on a change in pull request #301: Add markdown for
quota-based scheduling
URL: https://github.com/apache/helix/pull/301#discussion_r258743157
##
File path: website/0.8.3/src/site/markdown/quota_scheduling.md
##
@@ -0,0 +1,181 @@
+
+
+# Quota-based Task Sc
narendly commented on a change in pull request #301: Add markdown for
quota-based scheduling
URL: https://github.com/apache/helix/pull/301#discussion_r258758919
##
File path: website/0.8.3/src/site/markdown/quota_scheduling.md
##
@@ -0,0 +1,181 @@
+
+
+# Quota-based Task S
narendly commented on a change in pull request #301: Add markdown for
quota-based scheduling
URL: https://github.com/apache/helix/pull/301#discussion_r258758877
##
File path: website/0.8.3/src/site/markdown/quota_scheduling.md
##
@@ -0,0 +1,181 @@
+
+
+# Quota-based Task S
asfgit merged pull request #301: Add markdown for quota-based scheduling
URL: https://github.com/apache/helix/pull/301
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
narendly opened a new pull request #303: Add quota-based scheduling to tutorial
URL: https://github.com/apache/helix/pull/303
This is an automated message from the Apache Git Service.
To respond to the message, please log on
asfgit merged pull request #303: Add quota-based scheduling to tutorial
URL: https://github.com/apache/helix/pull/303
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub a
narendly opened a new pull request #304: Add quota-based scheduling to tutorial
URL: https://github.com/apache/helix/pull/304
This is an automated message from the Apache Git Service.
To respond to the message, please log on
asfgit merged pull request #304: Add quota-based scheduling to tutorial
URL: https://github.com/apache/helix/pull/304
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub a
narendly opened a new pull request #305: [HELIX-791] TASK2.0: Add RuntimeJobDag
with job iterator functionality
URL: https://github.com/apache/helix/pull/305
Job list iterator methods and underlying data structure were added to JobDag
to support retrieval of jobs by TaskDispatcher (to be i
narendly closed pull request #305: [HELIX-791] TASK2.0: Add RuntimeJobDag with
job iterator functionality
URL: https://github.com/apache/helix/pull/305
This is an automated message from the Apache Git Service.
To respond to
narendly opened a new pull request #306: [HELIX-791] TASK2.0: Add RuntimeJobDag
with job iterator functionality
URL: https://github.com/apache/helix/pull/306
Job list iterator methods and underlying data structure were added to JobDag
to support retrieval of jobs by TaskDispatcher (to be i
asfgit merged pull request #306: [HELIX-791] TASK2.0: Add RuntimeJobDag with
job iterator functionality
URL: https://github.com/apache/helix/pull/306
This is an automated message from the Apache Git Service.
To respond to th
narendly opened a new pull request #307: Push2
URL: https://github.com/apache/helix/pull/307
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to g
asfgit merged pull request #307: Minor improvements
URL: https://github.com/apache/helix/pull/307
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above
narendly opened a new pull request #308: Auto recovery maintenance feature
URL: https://github.com/apache/helix/pull/308
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHu
narendly closed pull request #308: Auto recovery maintenance feature
URL: https://github.com/apache/helix/pull/308
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and
narendly opened a new pull request #309: Auto-recovery maintenance mode feature
addition
URL: https://github.com/apache/helix/pull/309
This is an automated message from the Apache Git Service.
To respond to the message, plea
asfgit merged pull request #309: Auto-recovery maintenance mode feature addition
URL: https://github.com/apache/helix/pull/309
This is an automated message from the Apache Git Service.
To respond to the message, please log on
i3wangyi opened a new pull request #310: [HELIX-805] Implementation on
HelixAdmin to check if cluster in maint…
URL: https://github.com/apache/helix/pull/310
…enance mode
This is an automated message from the Apache Git Servi
asfgit merged pull request #310: [HELIX-805] Implementation on HelixAdmin to
check if cluster in maint…
URL: https://github.com/apache/helix/pull/310
This is an automated message from the Apache Git Service.
To respond to th
i3wangyi opened a new pull request #311: Check maitenanceMode rest api
implementation
URL: https://github.com/apache/helix/pull/311
Refactor the getClusterInfo method in ClusterAccessor rest api - Changes in
paused and maintenance mode
asfgit merged pull request #311: Check maitenanceMode rest api implementation
URL: https://github.com/apache/helix/pull/311
This is an automated message from the Apache Git Service.
To respond to the message, please log on G
narendly opened a new pull request #312: Minor bug fixes and improvements
URL: https://github.com/apache/helix/pull/312
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
i3wangyi opened a new pull request #313: [HELIX-813] HELIX instance level
validation util for rest API
URL: https://github.com/apache/helix/pull/313
This is an automated message from the Apache Git Service.
To respond to the
asfgit merged pull request #313: [HELIX-813] HELIX instance level validation
util for rest API
URL: https://github.com/apache/helix/pull/313
This is an automated message from the Apache Git Service.
To respond to the message
dasahcc commented on issue #312: Minor bug fixes and improvements
URL: https://github.com/apache/helix/pull/312#issuecomment-467695630
LGTM. But since this is large change. I will leave here for other reviewers
to review it. At same time. could you please rebase the head.
-
asfgit merged pull request #312: Minor bug fixes and improvements
URL: https://github.com/apache/helix/pull/312
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
narendly closed pull request #269: [HELIX-752] Add missing shutdown for
RoutingTableProvider
URL: https://github.com/apache/helix/pull/269
This is an automated message from the Apache Git Service.
To respond to the message,
narendly opened a new pull request #314: [HELIX-814] HELIX: Add back
ClusterDataCache for backward-compatibility
URL: https://github.com/apache/helix/pull/314
It was discovered that removing ClusterDataCache and changing public
interfaces (RebalanceStrategy, Rebalancer) caused backward-inc
asfgit merged pull request #314: [HELIX-814] HELIX: Add back ClusterDataCache
for backward-compatibility
URL: https://github.com/apache/helix/pull/314
This is an automated message from the Apache Git Service.
To respond to t
narendly opened a new pull request #315: Add markdown for auto-exit of
maintenance mode
URL: https://github.com/apache/helix/pull/315
This is an automated message from the Apache Git Service.
To respond to the message, pleas
dasahcc commented on issue #315: Add markdown for auto-exit of maintenance mode
URL: https://github.com/apache/helix/pull/315#issuecomment-469871837
Let's move user guide from FAQ to a separate section and describe it more on
operations.
narendly commented on issue #315: Add markdown for auto-exit of maintenance mode
URL: https://github.com/apache/helix/pull/315#issuecomment-469903213
Updated per your suggestions.
This is an automated message from the Apache G
dasahcc merged pull request #315: Add markdown for auto-exit of maintenance mode
URL: https://github.com/apache/helix/pull/315
This is an automated message from the Apache Git Service.
To respond to the message, please log on
narendly opened a new pull request #316: Revise the markdown and add more
context to how-to guide
URL: https://github.com/apache/helix/pull/316
This is an automated message from the Apache Git Service.
To respond to the mess
dasahcc merged pull request #316: Revise the markdown and add more context to
how-to guide
URL: https://github.com/apache/helix/pull/316
This is an automated message from the Apache Git Service.
To respond to the message, pl
bd2019us opened a new pull request #317: [HELIX-815] fix bug to avoid potential
crash
URL: https://github.com/apache/helix/pull/317
Fix: (HELIX-815-PATCH)[https://issues.apache.org/jira/browse/HELIX-815]
This is an automated
jiajunwang commented on issue #317: [HELIX-815] fix bug to avoid potential crash
URL: https://github.com/apache/helix/pull/317#issuecomment-478714480
Thanks for the fix. Merged to master branch.
This is an automated message fr
jiajunwang merged pull request #317: [HELIX-815] fix bug to avoid potential
crash
URL: https://github.com/apache/helix/pull/317
This is an automated message from the Apache Git Service.
To respond to the message, please log
bd2019us opened a new pull request #318: HELIX-816 use
System.currentTimeMillis()
URL: https://github.com/apache/helix/pull/318
new Date() is just a thin wrapper around System.currentTimeMillis(). Using
System.currentTimeMillis() can help speed up the system
--
narendly merged pull request #318: HELIX-816 use System.currentTimeMillis()
URL: https://github.com/apache/helix/pull/318
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
narendly commented on issue #318: HELIX-816 use System.currentTimeMillis()
URL: https://github.com/apache/helix/pull/318#issuecomment-482737721
Thanks for the PR :)
This is an automated message from the Apache Git Service.
To
alexgreenbar opened a new pull request #319: Fix long version of quickstart
guide of helix 0.8.4 release.
URL: https://github.com/apache/helix/pull/319
1. fix the wrong command parameter
2. add the command output
This is a
alexgreenbar opened a new pull request #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320
This is an automated message from the Apache Git Service.
To respond to the message, please log on
narendly commented on issue #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-490186774
Did it not compile on the old version?
This is an automated message from the Apache Git
jiajunwang commented on a change in pull request #319: Fix long version of
quickstart guide of helix 0.8.4 release.
URL: https://github.com/apache/helix/pull/319#discussion_r281770856
##
File path: website/0.8.4/src/site/markdown/Quickstart.md
##
@@ -406,7 +406,55 @@ Now,
alexgreenbar commented on a change in pull request #319: Fix long version of
quickstart guide of helix 0.8.4 release.
URL: https://github.com/apache/helix/pull/319#discussion_r282336490
##
File path: website/0.8.4/src/site/markdown/Quickstart.md
##
@@ -406,7 +406,55 @@ Now
alexgreenbar commented on issue #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-490741308
> Did it not compile on the old version?
Yes, I will paste more details.
Thi
alexgreenbar commented on a change in pull request #319: Fix long version of
quickstart guide of helix 0.8.4 release.
URL: https://github.com/apache/helix/pull/319#discussion_r282358381
##
File path: website/0.8.4/src/site/markdown/Quickstart.md
##
@@ -406,7 +406,55 @@ Now
alexgreenbar commented on issue #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-490770796
> Did it not compile on the old version?
I've provided more details on PR description.
--
alexgreenbar removed a comment on issue #320: Fix compile error on master
branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-490741308
> Did it not compile on the old version?
Yes, I will paste more details.
---
alexgreenbar commented on a change in pull request #319: Fix long version of
quickstart guide of helix 0.8.4 release.
URL: https://github.com/apache/helix/pull/319#discussion_r282336490
##
File path: website/0.8.4/src/site/markdown/Quickstart.md
##
@@ -406,7 +406,55 @@ Now
narendly commented on issue #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-491007976
I've tried on 2 different machines and it seems to work fine on both. I
don't have a problem with bumping up the dependency version, but at this point,
alexgreenbar commented on issue #320: Fix compile error on master branch.
URL: https://github.com/apache/helix/pull/320#issuecomment-491238706
I still got the same compile error after I did below actions
1. upgrade my maven from 3.3.1 to 3.6.1, which is the latest version
2. run comman
narendly opened a new issue #321: Issue testing
URL: https://github.com/apache/helix/issues/321
Test
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
narendly commented on issue #321: Issue testing
URL: https://github.com/apache/helix/issues/321#issuecomment-497111470
#320
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
narendly closed issue #321: Issue testing
URL: https://github.com/apache/helix/issues/321
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 g
jackjlli opened a new issue #322: java.lang.IllegalStateException: null is
thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322
When CRUSH based rebalance strategy algorithm is used, if the related
resource isn't enabled yet, Helix will throw `IllegalStateExcep
narendly opened a new issue #323: [HELIX-817] Support for multiple instance
group tags for Job config
URL: https://github.com/apache/helix/issues/323
Currently we can set only one instance group tag for a job.
jobCfg.setInstanceGroupTag("INSTANCEGROUPTAG");
It will be reall
jiajunwang commented on issue #323: [HELIX-817] Support for multiple instance
group tags for Job config
URL: https://github.com/apache/helix/issues/323#issuecomment-505100927
What's the difference between using multiple instance group tags and
defining a new tag that includes the desired
jiajunwang commented on issue #322: java.lang.IllegalStateException: null is
thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322#issuecomment-505104174
This is an error log. Could you please share the test case? What was
complained is lacking an eligible insta
jackjlli commented on issue #322: java.lang.IllegalStateException: null is
thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322#issuecomment-505109075
Hi @jiajunwang , basically we recently added a resource called
`leadControllerResource` in Pinot and set the
DImuthuUpe commented on issue #323: [HELIX-817] Support for multiple instance
group tags for Job config
URL: https://github.com/apache/helix/issues/323#issuecomment-505127003
@jiajunwang Good question. We can add tags to instances as much as we want.
But the problem is, is it scalable? I'm
jiajunwang commented on issue #323: [HELIX-817] Support for multiple instance
group tags for Job config
URL: https://github.com/apache/helix/issues/323#issuecomment-505162329
@DImuthuUpe, by supporting both sides configuration, we definitely cover
more use cases. I think it's very nice to
jiajunwang commented on issue #322: java.lang.IllegalStateException: null is
thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322#issuecomment-505180440
Thanks for the information, @jackjlli. So you are more concerned about the
verbose log, right? We can defini
jiajunwang edited a comment on issue #322: java.lang.IllegalStateException:
null is thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322#issuecomment-505180440
Thanks for the information, @jackjlli. So you are more concerned about the
verbose log, right? We can
jackjlli commented on issue #322: java.lang.IllegalStateException: null is
thrown when resource is disabled.
URL: https://github.com/apache/helix/issues/322#issuecomment-505184521
Great! Thanks for confirming!
This is an auto
narendly opened a new issue #324: Change logging level in CallbackHandler
URL: https://github.com/apache/helix/issues/324
Go through the logs and change some debug logs to info logs since they might
help with debugging.
This
jiajunwang opened a new issue #325: We should use system provided thread Id
instead of using a self maintained integer
URL: https://github.com/apache/helix/issues/325
https://github.com/apache/helix/blob/aa77f5c0bfc3cafb6482b1cef4a1689191cbd846/helix-core/src/main/java/org/apache/helix/tas
narendly opened a new issue #326: Add getWorkflow(long timeout) in TaskDriver
URL: https://github.com/apache/helix/issues/326
Sometimes ZK hangs in the getWorkflow() call, and we could provide an API
with a timeout to handle this at the application level.
AC:
1. Add getWorkflow(lo
narendly opened a new issue #327: Change the way we trigger rebalance in
purgeExpiredJobs
URL: https://github.com/apache/helix/issues/327
Currently, we use ResourceConfigs to trigger a rebalance. But this is
causing Helix to write a ResourceConfig back to ZK, which is not desirable when
a
dasahcc opened a new issue #328: Exclude ANY_INSTANCE for customized sibling
checks
URL: https://github.com/apache/helix/issues/328
Current Helix HealthCheck API checks the ANY_INSTANCE resources, which is
not necessary. Since ANY_INSTANCE resources only have single partition with 1
repli
dasahcc opened a new pull request #329: #328 Exclude ANY_INSTANCE for
customized sibling checks
URL: https://github.com/apache/helix/pull/329
Current Helix HealthCheck API checks the ANY_INSTANCE resources, which is
not necessary. Since ANY_INSTANCE resources only have single partition wit
dasahcc merged pull request #329: #328 Exclude ANY_INSTANCE for customized
sibling checks
URL: https://github.com/apache/helix/pull/329
This is an automated message from the Apache Git Service.
To respond to the message, ple
dasahcc closed issue #328: Exclude ANY_INSTANCE for customized sibling checks
URL: https://github.com/apache/helix/issues/328
This is an automated message from the Apache Git Service.
To respond to the message, please log on
dasahcc commented on issue #328: Exclude ANY_INSTANCE for customized sibling
checks
URL: https://github.com/apache/helix/issues/328#issuecomment-509931085
Fixed
This is an automated message from the Apache Git Service.
To res
lei-xia commented on issue #329: #328 Exclude ANY_INSTANCE for customized
sibling checks
URL: https://github.com/apache/helix/pull/329#issuecomment-510227742
5 tests fails, should we fix the tests?
This is an automated messag
dasahcc commented on issue #329: #328 Exclude ANY_INSTANCE for customized
sibling checks
URL: https://github.com/apache/helix/pull/329#issuecomment-510239011
> 5 tests fails, should we fix the tests?
There is only 1 test is unstable. We will try to fix it. I run it locally
again, th
dasahcc opened a new issue #330: AutoFallbackPropertyStore make cache path to
be null
URL: https://github.com/apache/helix/issues/330
Current AutoFallbackPropertyStore instantiation mark the zkCachePath to be
null. So the caching functionality is not work for this type of ProertyStore.
dasahcc commented on issue #327: Change the way Helix triggers rebalance in
purgeExpiredJobs
URL: https://github.com/apache/helix/issues/327#issuecomment-510685974
There is a race condition if user update the ResourceConfig at same time.
Say original p1 has preference list: [1, 2, 3]. Heli
Jackie-Jiang opened a new issue #331: CallbackHandler async re-subscribe
watcher can potentially miss events
URL: https://github.com/apache/helix/issues/331
In CallbackHandler, the re-subscription of the watcher for CALLBACK happens
asynchronously. If the re-subscription happens after the
i3wangyi opened a new issue #332: Issue when client ONLY sets cluster level ANY
throttle config
URL: https://github.com/apache/helix/issues/332
The current implementation has a loophole when
1. When client **ONLY** sets cluster level ANY throttle config (no specific
type throttle)
2
i3wangyi opened a new pull request #333: Fix issue when client only sets ANY at
cluster level throttle config
URL: https://github.com/apache/helix/pull/333
**Issues**
(fixes #332 Issue when client ONLY sets cluster level ANY throttle config )
**Description**
The fix will decrem
dasahcc opened a new issue #334: ZNRecord updater merge listfield with adding
duplicate elements
URL: https://github.com/apache/helix/issues/334
Current HelixDataAccesor updateProperty uses ZNRecordUpdater. It's merge
logic just simply adding all elements when do a merge for ZNRecord. That
i3wangyi opened a new issue #335: Support Property Read API - helix-rest
URL: https://github.com/apache/helix/issues/335
Currently, there's no endpoint available for users to read propertystore
content directly via helix-rest endpoint. Users have to rely on tools to view
the data on zookee
dasahcc opened a new pull request #336: #334 Fix invoke rebalance by "touching"
IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336
**Issues**
(fixes #334 )
**Description**
Current HelixDataAccesor updateProperty uses ZNRecordUpdater. It's merge
logic jus
jiajunwang commented on a change in pull request #336: #334 Fix invoke
rebalance by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r303739444
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
@@ -10
jiajunwang commented on a change in pull request #336: #334 Fix invoke
rebalance by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r303740523
##
File path:
helix-core/src/main/java/org/apache/helix/controller/rebalancer/util/RebalanceS
dasahcc commented on a change in pull request #336: #334 Fix invoke rebalance
by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r303745274
##
File path:
helix-core/src/main/java/org/apache/helix/controller/rebalancer/util/RebalanceSche
alirezazamani opened a new pull request #337: Fixing IllegalStateException for
CRUSH based rebalance strategy algorithm.
URL: https://github.com/apache/helix/pull/337
In CRUSH based rebalance strategy, when there is no eligible instance for
the controller to use, it shows an exception whic
lei-xia commented on a change in pull request #336: #334 Fix invoke rebalance
by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r304025056
##
File path:
helix-core/src/main/java/org/apache/helix/controller/rebalancer/util/RebalanceSche
lei-xia commented on a change in pull request #336: #334 Fix invoke rebalance
by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r304026414
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
@@ -1021,
lei-xia commented on a change in pull request #336: #334 Fix invoke rebalance
by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r304026926
##
File path:
helix-core/src/test/java/org/apache/helix/util/TestRebalanceScheduler.java
##
dasahcc commented on a change in pull request #336: #334 Fix invoke rebalance
by "touching" IdealState/ResourceConfig
URL: https://github.com/apache/helix/pull/336#discussion_r304050227
##
File path:
helix-core/src/test/java/org/apache/helix/util/TestRebalanceScheduler.java
##
1 - 100 of 706 matches
Mail list logo
