Github user revans2 commented on a diff in the pull request:
https://github.com/apache/storm/pull/2836#discussion_r218174140
--- Diff:
storm-server/src/main/java/org/apache/storm/daemon/nimbus/Nimbus.java ---
@@ -4624,7 +4569,7 @@ public boolean isRemoteBlobExists(String blobKey)
throws AuthorizationException,
return true;
}
- private static final class Assoc<K, V> implements UnaryOperator<Map<K,
V>> {
+ static final class Assoc<K, V> implements UnaryOperator<Map<K, V>> {
--- End diff --
Yes, I'll make them private again. This was because as I was refactoring I
kept the Assoc and Dissoc until the very end when I fixed the threading.
---