Github user revans2 commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2711#discussion_r196848589
  
    --- Diff: storm-server/src/main/java/org/apache/storm/Testing.java ---
    @@ -660,8 +661,8 @@ public static Tuple testTuple(List<Object> values, 
MkTupleParam param) {
                 }
             }
     
    -        Map<Integer, String> taskToComp = new HashMap<>();
    -        taskToComp.put(task, component);
    +        CustomIndexArray<String> taskToComp = new 
CustomIndexArray<>(task,task+1);
    --- End diff --
    
    If lower and upper are inclusive why do we need task, task+1?


---

Reply via email to