Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2566#discussion_r170359655
--- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java ---
@@ -1559,8 +1577,8 @@ public static boolean isLocalhostAddress(String
address) {
Set<Integer> ids = srcMap.keySet();
Integer largestId = ids.stream().max(Integer::compareTo).get();
int end = largestId - start;
- ArrayList<V> result = new ArrayList<>(Collections.nCopies(end+1 ,
null)); // creates array[largestId+1] filled with nulls
- for( Map.Entry<Integer, V> entry : srcMap.entrySet() ) {
+ ArrayList<V> result = new ArrayList<>(Collections.nCopies(end + 1,
null)); // creates array[largestId+1] filled with nulls
--- End diff --
Auto-corrected via IntelliJ. Let me revert if we don't want to fix
unrelated thing.
---
