Github user maoling commented on a diff in the pull request:
https://github.com/apache/zookeeper/pull/495#discussion_r177294282
--- Diff:
src/java/main/org/apache/zookeeper/server/ReferenceCountedACLCache.java ---
@@ -109,16 +109,18 @@ public synchronized void deserialize(InputArchive ia)
throws IOException {
}
List<ACL> aclList = new ArrayList<ACL>();
Index j = ia.startVector("acls");
- while (!j.done()) {
- ACL acl = new ACL();
- acl.deserialize(ia, "acl");
- aclList.add(acl);
- j.incr();
+ if (j != null) {
--- End diff --
- can we have a more elegant way to process this NPE ?
- BTW. incorrect commit message. `git commit --amend -m "your new
message"` to modify it
---
