You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@zookeeper.apache.org by GitBox <gi...@apache.org> on 2021/07/07 08:38:56 UTC

[GitHub] [zookeeper] autumind edited a comment on pull request #1719: ZOOKEEPER-4325: Fix bug when list "/" with ZkUtil::listSubTreeBFS

autumind edited a comment on pull request #1719:
URL: https://github.com/apache/zookeeper/pull/1719#issuecomment-875409050


   > empty line at the end of the file
   
   
   > @autumind
   > You really have an empty line at the end of the file: `ZKUtil.java`. After you revert it, I will merge it asap at this weekend(07-11).
   > 
   > ```
   > git  diff 70f70d821c2
   > 
   > @@ -290,4 +291,4 @@ public static String aclToString(List<ACL> acls) {
   >          }
   >          return sb.toString();
   >      }
   > -}
   > \ No newline at end of file
   > +}
   > diff --git a/zookeeper-server/src/test/java/org/apache/zookeeper/ZKUtilTest.java b/zookeeper-server/src/test/java/org/apache/zookeeper/ZKUtilTest.java
   > index 1111c2a23..3ad1ed94f 100644
   > ```
   What a silly mistake, I've reverted 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: notifications-unsubscribe@zookeeper.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org