[GitHub] [zookeeper] lvfangmin commented on issue #1138: ZOOKEEPER-3546 - Allow optional deletion of never used Container Nodes

[GitBox <gi...@apache.org>]

lvfangmin commented on issue #1138: ZOOKEEPER-3546 - Allow optional deletion of never used Container Nodes
URL: https://github.com/apache/zookeeper/pull/1138#issuecomment-554002507
 
 
   > This is because the initial implementation does not delete container nodes with a cversion of 0
   
   It seems to me we're trying to add a 'TTL' for the container besides the behavior of deleting the container node when the children are all deleted. It seems changed the semantic of this API, and the 'maxNeverUsedIntervalMs' setting is hard to define, set it too small may cause problem for the users. Also, different users may have different expectations on that, have a single value may not work for multi-tenant ensemble.
   
   I also have the same question as eolivelli, if the client crashed, they'll try to create the same container, right? Are they're creating some unique container name every time?

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services