[GitHub] storm issue #2143: STORM-2503: Restore comparator logic in `DefaultResourceA...

2017-05-31 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/2143 @jerrypeng Like I said in the previous pull request. The unit test that failed was not cause by this change. It was cause by an actual fix to the comparison logic for `sortComponents`. In my

[GitHub] storm issue #2143: STORM-2503: Restore comparator logic in `DefaultResourceA...

2017-05-29 Thread jerrypeng
Github user jerrypeng commented on the issue: https://github.com/apache/storm/pull/2143 @adityasharad Thanks for changing it back. What about the unit test that was modified as well? --- If your project is set up for it, you can reply to this email and have your reply appear on