Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/869#issuecomment-155431246
@ashnazg the test "might" "logically" pass. The code is a bit complex.
When debugging these in the past the tests use simulated time, which is a great
thing it lets us run tests that would take a long time very quickly, but to
make it work properly there are some critical points where we wait for the
simulated cluster to be idle. Typically we see timeouts happen here, waiting
for the entire cluster to stop. But for some reason on really slow hardware,
that I don't totally understand yet, this does not always happen. and
increasing the timeout does not always induce it to be in that state.
I am +1 on merging this in, but I am not totally sure that it will "fix"
the problem.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
