keith-turner commented on code in PR #5286: URL: https://github.com/apache/accumulo/pull/5286#discussion_r1943192998
########## core/src/main/java/org/apache/accumulo/core/zookeeper/ZcNode.java: ########## @@ -29,27 +29,42 @@ * and children are obtained from zookeeper at different times. This class is structured so that * data can be obtained first and then children added later or visa veras. * - * <p> - * Four distinct states can be cached for a zookeeper node. - * <ul> - * <li>Can cache that a node does not exist in zookeeper. This state is represented by data, state, - * and children all being null.</li> - * <li>Can cache only the data for a zookeeper node. For this state data and stat are non-null while - * children is null. Calling getChildren on node in this state will throw an exception.</li> - * <li>Can cache only the children for a zookeeper node. For this state children is non-null while - * data and stat are null. Calling getData or getStat on node in this state will throw an - * exception.</li> Review Comment: I updated the javadoc for the methods to be a more specific. -- 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]
