NealSun96 edited a comment on issue #727: Add MetadataStoreRoutingDataWriter
with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584335976
> > A generic comment on all writer logics: some deletion operations can
lead to a data state that is deemed
NealSun96 commented on issue #727: Add MetadataStoreRoutingDataWriter with
DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584335976
> > A generic comment on all writer logics: some deletion operations can
lead to a data state that is deemed "invalid"
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377260454
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377259958
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377311433
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377261566
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377318169
##
File path:
narendly commented on a change in pull request #741: Fix the possibile
ConcurrentMod exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377351837
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
narendly commented on a change in pull request #741: Fix the possibile
ConcurrentMod exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377351141
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377320038
##
File path:
NealSun96 commented on issue #731: Add TrieRoutingData constructor
URL: https://github.com/apache/helix/pull/731#issuecomment-584374793
This PR is ready to be merged, approved by @narendly
Final commit message:
## Add TrieRoutingData constructor ##
This PR adds a constructor to
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377381252
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -124,8 +128,91
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377289102
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/ZkMetadataStoreDirectory.java
##
@@
narendly commented on issue #727: Add MetadataStoreRoutingDataWriter with
DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584308561
> A generic comment on all writer logics: some deletion operations can lead
to a data state that is deemed "invalid" by
jiajunwang opened a new issue #739: Error log about WAGED rebalancer missing is
not necessary
URL: https://github.com/apache/helix/issues/739
This only appears in the test environment, the log is not necessarily to be
an error log.
10588 [main] ERROR
jiajunwang opened a new pull request #740: Change the WAGED rebalancer missing
error log to warning.
URL: https://github.com/apache/helix/pull/740
### Issues
- [ ] My PR addresses the following Helix issues and references them in the
PR description:
#739
###
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377359734
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLockInfo.java
##
@@ -0,0 +1,92 @@
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377353944
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLock.java
##
@@ -0,0 +1,198 @@
+/*
+
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377358921
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLockInfo.java
##
@@ -0,0 +1,92 @@
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377352927
##
File path: helix-lock/src/main/java/org/apache/helix/lock/HelixLockScope.java
##
@@ -0,0 +1,179 @@
+/*
+ *
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377356770
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLock.java
##
@@ -0,0 +1,198 @@
+/*
+
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377357255
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLock.java
##
@@ -0,0 +1,198 @@
+/*
+
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377358330
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLock.java
##
@@ -0,0 +1,198 @@
+/*
+
jiajunwang commented on a change in pull request #718: Implement Helix
nonblocking lock
URL: https://github.com/apache/helix/pull/718#discussion_r377359284
##
File path:
helix-lock/src/main/java/org/apache/helix/lock/ZKHelixNonblockingLockInfo.java
##
@@ -0,0 +1,92 @@
NealSun96 edited a comment on issue #727: Add MetadataStoreRoutingDataWriter
with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584335976
> > A generic comment on all writer logics: some deletion operations can
lead to a data state that is deemed
alirezazamani opened a new issue #738: ConcurrentModificationException for the
TaskGarbageCollectionStage
URL: https://github.com/apache/helix/issues/738
In the async TaskGarbageCollectionStage, since the workflow context can be
changes in other threads (main pipeline), while looping
jiajunwang closed issue #734: testRebalancerReset is not stable
URL: https://github.com/apache/helix/issues/734
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
NealSun96 commented on issue #731: Add TrieRoutingData constructor
URL: https://github.com/apache/helix/pull/731#issuecomment-584413855
This PR is ready to be merged, approved by @narendly
Final commit message:
## Add TrieRoutingData constructor ##
This PR adds a constructor to
NealSun96 removed a comment on issue #731: Add TrieRoutingData constructor
URL: https://github.com/apache/helix/pull/731#issuecomment-584374793
This PR is ready to be merged, approved by @narendly
Final commit message:
## Add TrieRoutingData constructor ##
This PR adds a
jiajunwang closed issue #739: Error log about WAGED rebalancer missing is not
necessary
URL: https://github.com/apache/helix/issues/739
This is an automated message from the Apache Git Service.
To respond to the message,
alirezazamani opened a new pull request #741: Fix the possibile ConcurrentMod
exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741
### Issues
- [x] My PR addresses the following Helix issues and references them in the
PR title:
Fixes #738
jiajunwang merged pull request #740: Change the WAGED rebalancer missing error
log to warning.
URL: https://github.com/apache/helix/pull/740
This is an automated message from the Apache Git Service.
To respond to the
jiajunwang commented on issue #740: Change the WAGED rebalancer missing error
log to warning.
URL: https://github.com/apache/helix/pull/740#issuecomment-584373317
This PR is ready to be merged, approved by @alirezazamani
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377340478
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377341015
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377341935
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377342252
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377378255
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377378006
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377356452
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377321200
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377370447
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377373675
##
File path:
dasahcc commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377373000
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377395026
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377394933
##
File path:
narendly commented on a change in pull request #727: Add
MetadataStoreRoutingDataWriter with DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#discussion_r377393176
##
File path:
JLLeitschuh opened a new pull request #742: [SECURITY] Use HTTPS to resolve
dependencies in Maven Build
URL: https://github.com/apache/helix/pull/742
pkuwm opened a new pull request #744: [WIP] Add read endpoints of metadata
store directory service to helix rest
URL: https://github.com/apache/helix/pull/744
### Issues
- [ ] My PR addresses the following Helix issues and references them in the
PR description:
Implements
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377461380
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377462232
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377461331
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377460200
##
File path:
narendly commented on a change in pull request #741: Fix ConcurrentModification
exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377423574
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
@@
pkuwm opened a new issue #743: Add read REST endpoints of metadata store
directory service to helix rest
URL: https://github.com/apache/helix/issues/743
We have metadata store directory service to help scale out zookeeper.
Metadata store directory service provides REST APIs to access.
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377422265
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377422061
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377421981
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377420914
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377421482
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377421852
##
File path:
narendly commented on a change in pull request #744: [WIP] Add read endpoints
of metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377422440
##
File path:
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377433026
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
narendly commented on a change in pull request #741: Fix ConcurrentModification
exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377434390
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
@@
pkuwm commented on a change in pull request #744: [WIP] Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377428667
##
File path:
pkuwm commented on a change in pull request #744: [WIP] Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377453982
##
File path:
pkuwm commented on a change in pull request #744: [WIP] Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377429155
##
File path:
pkuwm commented on a change in pull request #744: [WIP] Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377428371
##
File path:
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377433026
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
narendly commented on issue #727: Add MetadataStoreRoutingDataWriter with
DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584431516
This PR is ready to be merged, approved by @dasahcc .
Add MetadataStoreRoutingDataWriter with
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377418194
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377418237
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377418263
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377462688
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377462037
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377460300
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377462456
##
File path:
narendly commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377462764
##
File path:
narendly merged pull request #731: Add TrieRoutingData constructor
URL: https://github.com/apache/helix/pull/731
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
narendly merged pull request #727: Add MetadataStoreRoutingDataWriter with
DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727
This is an automated message from the Apache Git Service.
To respond to the
dasahcc commented on issue #727: Add MetadataStoreRoutingDataWriter with
DistributedLeaderElection
URL: https://github.com/apache/helix/pull/727#issuecomment-584429052
Remember to resolve the conflict
This is an automated
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377467457
##
File path:
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377471595
##
File path:
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377470116
##
File path:
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377468335
##
File path:
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377467809
##
File path:
pkuwm commented on a change in pull request #744: Add read endpoints of
metadata store directory service to helix rest
URL: https://github.com/apache/helix/pull/744#discussion_r377468093
##
File path:
narendly opened a new pull request #745: [WIP] Create RealmAwareZkClientFactory
interface
URL: https://github.com/apache/helix/pull/745
This is an automated message from the Apache Git Service.
To respond to the message,
alirezazamani commented on a change in pull request #741: Fix
ConcurrentModification exception in Workflow Garbage Collection
URL: https://github.com/apache/helix/pull/741#discussion_r377436725
##
File path: helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
##
pkuwm commented on issue #744: Add read endpoints of metadata store directory
service to helix rest
URL: https://github.com/apache/helix/pull/744#issuecomment-584512780
@narendly Thanks for reviewing! Addressed your comments and updated the PR.
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377210368
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -37,15 +39,26
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377210986
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -37,15 +39,26
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377213249
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -58,20 +71,26
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377216613
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -88,72 +107,174
NealSun96 commented on a change in pull request #731: Add TrieRoutingData
constructor
URL: https://github.com/apache/helix/pull/731#discussion_r377216518
##
File path:
helix-rest/src/main/java/org/apache/helix/rest/metadatastore/TrieRoutingData.java
##
@@ -88,72 +107,174
1 - 100 of 113 matches
Mail list logo