[GitHub] zookeeper pull request #590: [ZOOKEEPER-1177] Add the memory optimized watch...

[hanm]

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

    https://github.com/apache/zookeeper/pull/590#discussion_r219640916
  
    --- Diff: src/java/main/org/apache/zookeeper/server/DataTree.java ---
    @@ -253,6 +259,14 @@ public DataTree() {
             addConfigNode();
     
             nodeDataSize.set(approximateDataSize());
    +        try {
    +            dataWatches = WatchManagerFactory.createWatchManager();
    +            childWatches = WatchManagerFactory.createWatchManager();
    +        } catch (Exception e) {
    +            LOG.error("Unexpected exception when creating WatchManager, " +
    +                    "exiting abnormally", e);
    --- End diff --
    
    nit: use parameterized logging here.


---