Github user njhill commented on a diff in the pull request:
    --- Diff: 
    @@ -704,8 +687,7 @@ private TreeNode find(String findPath)
                     TreeNode childNode = entry.getValue();
                     ChildData childData = childNode.childData;
    -                // Double-check liveness after retreiving data.
    --- End diff --
    The reason I removed it is that unlike before it's not really "double" 
checking anymore against the separate `nodeState` field, there's only a single 
check for liveness which is now implicit in the value of `childData`. But I 
could well have misinterpreted the reason for comment, no problem with leaving 
it in!


Reply via email to