zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From phunt <...@git.apache.org>
Subject [GitHub] zookeeper pull request #377: [ZOOKEEPER-2901] TTL Nodes don't work with Serv...
Date Fri, 20 Oct 2017 23:41:25 GMT
Github user phunt commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/377#discussion_r146087059
  
    --- Diff: conf/zoo_sample.cfg ---
    @@ -6,6 +6,10 @@ initLimit=10
     # The number of ticks that can pass between 
     # sending a request and getting an acknowledgement
     syncLimit=5
    +# enable TTL Nodes
    +# IMPORTANT: when enabled, your server ID cannot be greater than 254
    +# See https://zookeeper.apache.org/doc/r3.5.3-beta/zookeeperAdmin.html#sc_configuration
for details
    +ttlNodesEnabled=false
    --- End diff --
    
    I was actually thinking that this specific line (not in general) should be true by default.
That way anyone starting out would get the feature enabled by default - and they wouldn't
have any legacy serverids specified. What do you think?


---

Mime
View raw message