Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2433#discussion_r152894349
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -231,7 +183,15 @@
@VisibleForTesting
public static final List<ACL> ZK_ACLS =
Arrays.asList(ZooDefs.Ids.CREATOR_ALL_ACL.get(0),
- new ACL(ZooDefs.Perms.READ | ZooDefs.Perms.CREATE,
ZooDefs.Ids.ANYONE_ID_UNSAFE));
+ new ACL(ZooDefs.Perms.READ ^ ZooDefs.Perms.CREATE,
ZooDefs.Ids.ANYONE_ID_UNSAFE));
--- End diff --
Old nimbus.clj uses `bit-or` not `bit-xor`.
https://github.com/apache/storm/blob/1.x-branch/storm-core/src/clj/org/apache/storm/daemon/nimbus.clj#L147
https://github.com/apache/storm/blob/1.1.x-branch/storm-core/src/clj/org/apache/storm/daemon/nimbus.clj#L147
---
