Github user zd-project commented on a diff in the pull request:
https://github.com/apache/storm/pull/2763#discussion_r203769152
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -2809,21 +2809,19 @@ public void launchServer() throws Exception {
}
});
+ //Be cautious using method reference instead of lambda.
subexpression preceding :: will be evaluated only upon evaluation
--- End diff --
Sorry I was writing this to myself because I was not super familiar with
Java lambda. Should probably delete it as it's not relevant to code.
---
