[ 
https://issues.apache.org/jira/browse/CURATOR-477?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16624961#comment-16624961
 ] 

ASF GitHub Bot commented on CURATOR-477:
----------------------------------------

Github user dragonsinth commented on a diff in the pull request:

    https://github.com/apache/curator/pull/278#discussion_r219688434
  
    --- Diff: 
curator-recipes/src/main/java/org/apache/curator/framework/recipes/cache/TreeCache.java
 ---
    @@ -84,6 +85,7 @@
             private ExecutorService executorService = null;
             private int maxDepth = Integer.MAX_VALUE;
             private boolean createParentNodes = false;
    +        private boolean createZkWatches = true;
    --- End diff --
    
    I'd probably reverse this as "disableZkWatch" since that's the unusual case.


> Ability to turn off Zk Watches in Curator Framework
> ---------------------------------------------------
>
>                 Key: CURATOR-477
>                 URL: https://issues.apache.org/jira/browse/CURATOR-477
>             Project: Apache Curator
>          Issue Type: Improvement
>          Components: Framework
>    Affects Versions: 4.0.1
>            Reporter: Rama Chavali
>            Priority: Major
>
> In our use case, we use *{{TreeCache}}* to get Zk Data periodically. We start 
> *{{TreeCache}}* read data and close it. In this use case, The 
> {{ZkWatchManager}} of {{ZooKeeper}} class keeps growing for every TreeCache 
> operation because new {{TreeNode}} objects are created and added there 
> leading to a memory leak. Also since we do not want the Watcher to 
> periodically watch, this creates unnecessary background operations.
> Can we introduce a builder flag in CuratorFramework's Builder some thing 
> called "createZkWatches" that we can use to turn the watchers off? The 
> default would be set to true to retain the current behaviour.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to