Github user Ethanlm commented on a diff in the pull request:
https://github.com/apache/storm/pull/2548#discussion_r166111464
--- Diff:
storm-client/src/jvm/org/apache/storm/daemon/worker/WorkerState.java ---
@@ -20,6 +20,17 @@
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.Sets;
+import java.io.IOException;
+import java.nio.ByteBuffer;
+import java.util.*;
--- End diff --
I believe storm community says better to not use *.
Btw, in Intellij, you can navigate Preference--> Editor --> code style -->
Java and set `class count to use import with *` to a high number to avoid auto
collapse
---