Github user revans2 commented on the issue:
https://github.com/apache/storm/pull/2700
The travis failure looks unrelated.
---
Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2700
@danny0405 No it's fine. Thanks for the contribution.
---
Github user danny0405 commented on the issue:
https://github.com/apache/storm/pull/2700
@Ethanlm
Sorry i have not did the benchmark test, but we can see the promotion
through the storm-core module building time, it's about 10 seconds faster on my
laptop.
I can add in the
Github user Ethanlm commented on the issue:
https://github.com/apache/storm/pull/2700
Looks good. Do you have any performance test results to share so that we
can have an idea how much performance gain this patch gets? It's fine if you
don't have it
---