You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2021/08/19 08:07:39 UTC

[GitHub] [zookeeper] yuMoyu opened a new pull request #1738: Update zookeeperTutorial.md

yuMoyu opened a new pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738


   With EPHEMERAL_SEQUENTIAL,the generated node name will be added with the sequence number, and the delete() will report error that it cannot be found
   EPHEMERAL_SEQUENTIAL产生的节点名会加上顺序编号,删除会报找不到


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] asfgit closed pull request #1738: ZOOKEEPER-4406: fix the znode type for Barrier implementation in the zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738


   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] maoling closed pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
maoling closed pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738


   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] yuMoyu commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
yuMoyu commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-909800966


   > Hi,
   > I'm thinking about to implement ZooKeeper Server.
   > 
   > Want to configure the server as if Same data is received to update for a node then system should ignore this update request.
   > Kindly assist me for the same.
   
   When using the class of Zookeeper,maybe you can call getData() before setData()?


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] khakindrad commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
khakindrad commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-909131645


   Hi,
       I'm thinking about to implement ZooKeeper Server. 
   
      Want to configure the server as if Same data is received to update for a node then system should ignore this update request.
    Kindly assist me for the same.


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] maoling commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
maoling commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-905394236


   - Good catch. @yuMoyu Thanks for this contribution. It's better to create a [JIRA](https://issues.apache.org/jira/projects/ZOOKEEPER/) issue (sign up JIRA if you don't have an account) to bind this PR to a JIRA-ID. ZooKeeper uses the GitHub workflow. The contributor guideline is [here](https://cwiki.apache.org/confluence/display/ZOOKEEPER/HowToContribute)
   - please write the GitHub description with English, other than Chinese
   - I believe that the implementation of barrier must have other places to improve. Could you please try to improve it?
   the following is a good references:
       - https://curator.apache.org/curator-recipes/barrier.html
       - https://github.com/apache/curator/blob/master/curator-recipes/src/main/java/org/apache/curator/framework/recipes/barriers/DistributedBarrier.java#L70
   
   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] khakindrad commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
khakindrad commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-909131645


   Hi,
       I'm thinking about to implement ZooKeeper Server. 
   
      Want to configure the server as if Same data is received to update for a node then system should ignore this update request.
    Kindly assist me for the same.


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] maoling commented on pull request #1738: ZOOKEEPER-4406: fix the znode type for Barrier implementation in the zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
maoling commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-955643095


   @yuMoyu 
   Thanks for your contribution.


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] yuMoyu commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
yuMoyu commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-909800966


   > Hi,
   > I'm thinking about to implement ZooKeeper Server.
   > 
   > Want to configure the server as if Same data is received to update for a node then system should ignore this update request.
   > Kindly assist me for the same.
   
   When using the class of Zookeeper,maybe you can call getData() before setData()?


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] yuMoyu commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
yuMoyu commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-906235870


   > * Good catch. @yuMoyu Thanks for this contribution. It's better to create a [JIRA](https://issues.apache.org/jira/projects/ZOOKEEPER/) issue (sign up JIRA if you don't have an account) to bind this PR to a JIRA-ID. ZooKeeper uses the GitHub workflow. The contributor guideline is [here](https://cwiki.apache.org/confluence/display/ZOOKEEPER/HowToContribute)
   > * please write the GitHub description with English, other than Chinese
   > * I believe that the implementation of barrier must have other places to improve. Could you please try to improve it?
   >   the following is a good references:
   >   
   >   * https://curator.apache.org/curator-recipes/barrier.html
   >   * https://github.com/apache/curator/blob/master/curator-recipes/src/main/java/org/apache/curator/framework/recipes/barriers/DistributedBarrier.java#L70
   
   Sorry, I don't quite understand which of the following you mean. 1、 The improvements I mentioned can be optimized. 2、 There are other aspects that can be improved in this document. 3、 the classes in the link can be improved.


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] maoling commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
maoling commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-911549902


   > Sorry, I don't quite understand which of the following you mean. 1、 The improvements I mentioned can be optimized. 2、 There are other aspects that can be improved in this document. 3、 the classes in the link can be improved.
   
   @yuMoyu  What I want express is: the `barrier` implementation in the zk doc is not good enough and there are other aspects that can be improved in this doc. The reference link I provided may give you some inspirations.
   


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zookeeper] khakindrad commented on pull request #1738: Update zookeeperTutorial.md

Posted by GitBox <gi...@apache.org>.
khakindrad commented on pull request #1738:
URL: https://github.com/apache/zookeeper/pull/1738#issuecomment-909131645


   Hi,
       I'm thinking about to implement ZooKeeper Server. 
   
      Want to configure the server as if Same data is received to update for a node then system should ignore this update request.
    Kindly assist me for the same.


-- 
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 go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org