PapaCharlie commented on code in PR #1950: URL: https://github.com/apache/zookeeper/pull/1950#discussion_r1204513349
########## zookeeper-server/src/main/java/org/apache/zookeeper/WatchedEvent.java: ########## @@ -31,27 +31,38 @@ */ @InterfaceAudience.Public public class WatchedEvent { + public static final long NO_ZXID = -1L; private final KeeperState keeperState; private final EventType eventType; - private String path; + private final String path; + private final long zxid; /** - * Create a WatchedEvent with specified type, state and path + * Create a WatchedEvent with specified type, state, path and zxid */ - public WatchedEvent(EventType eventType, KeeperState keeperState, String path) { + public WatchedEvent(EventType eventType, KeeperState keeperState, String path, long zxid) { Review Comment: This is preserved below, it's just not marked as @deprecated because there are many uses of WatchedEvent that correctly do not have a zxid. -- 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: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org