rabashizade commented on code in PR #2082:
URL: https://github.com/apache/helix/pull/2082#discussion_r870660486


##########
zookeeper-api/src/main/java/org/apache/helix/zookeeper/zkclient/ZkClient.java:
##########
@@ -85,6 +86,7 @@
 public class ZkClient implements Watcher {
   private static final Logger LOG = LoggerFactory.getLogger(ZkClient.class);
 
+  public static final long DEFAULT_TTL = -1L;

Review Comment:
   Yes, that's a better name. I'll change it.
   Also yes, it will be used in other classes such as BaseDataAccessors and 
ZkClient implementations. I was thinking of creating a separate class for this 
constant, but since it was only one variable I thought I'd just put it in 
ZkClient. Let me know if a separate constants class is preferred and I can add 
it.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to