[GitHub] storm issue #2558: [STORM-2950] fix ArrayIndexOutOfBoundsException in Fields...

2018-02-14 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2558 @Ethanlm `Fixing ArrayIndexOutOfBoundsException` is the side effect of making the behavior back to 1.x version. We already know 1.x version doesn't have such issue, so it can be placed to side no

[GitHub] storm issue #2558: [STORM-2950] fix ArrayIndexOutOfBoundsException in Fields...

2018-02-14 Thread Ethanlm
Github user Ethanlm commented on the issue: https://github.com/apache/storm/pull/2558 @HeartSaVioR Will this be to long: [STORM-2950] fix ArrayIndexOutOfBoundsException in FieldsGrouper.chooseTasks function and make it consistent with 1.x version ---

[GitHub] storm issue #2558: [STORM-2950] fix ArrayIndexOutOfBoundsException in Fields...

2018-02-14 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/2558 @Ethanlm Thanks for applying review comment so quickly! Now the meaning of patch would be very different from issue summary & description (as well as PR's title and commit message) which