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 2022/06/28 16:07:11 UTC

[GitHub] [zookeeper] tisonkun commented on pull request #1820: ZOOKEEPER-4475: Fix NodeChildrenChanged delivered to recursive watcher

tisonkun commented on PR #1820:
URL: https://github.com/apache/zookeeper/pull/1820#issuecomment-1168917833

   Some other concerns:
   
   1. The description in this patch is different from JIRA's content:
   
   > This requires server to never deliver NodeChildrenChanged for node's descendants.
   
   In this patch we filter event on the client side, which has nothing to do with server side logics.
   
   2. Do you have a real world workload needs this patch? I'm glad to see a concrete use case for PersistentRecursiveWatch and how the behavior changed here affects the application logic.


-- 
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