Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2908#discussion_r236360412
--- Diff: storm-server/src/main/java/org/apache/storm/LocalCluster.java ---
@@ -323,6 +342,13 @@ private static boolean areAllWorkersWaiting() {
}
}
+ private static final long DEFAULT_ZK_PORT = 2181;
--- End diff --
Nit: Please move this to the other constants so it's not in the middle of
method declarations.---
