Github user d2r commented on the pull request:
https://github.com/apache/incubator-storm/pull/38#issuecomment-47559759
@dashengju
OK, can you try with this patch?
```Diff
diff --git a/storm-core/src/clj/backtype/storm/testing.clj
b/storm-core/src/clj/backtype/storm/testing.clj
index c3eb76f..70c783a 100644
--- a/storm-core/src/clj/backtype/storm/testing.clj
+++ b/storm-core/src/clj/backtype/storm/testing.clj
@@ -234,7 +234,10 @@
(log-error t# "Error in cluster")
(throw t#))
(finally
- (kill-local-storm-cluster ~cluster-sym)))))
+ (let [keep-waiting?# (atom true)]
+ (future (while @keep-waiting?# (simulate-wait ~cluster-sym)))
+ (kill-local-storm-cluster ~cluster-sym)
+ (reset! keep-waiting?# false))))))
(defmacro with-simulated-time-local-cluster
[& args]
```
The above seems to fix the issue. It could prevent spurious hangs we see
with other tests as well.
> -(def TEST-TIMEOUT-MS 5000)
> +(def TEST-TIMEOUT-MS 30000)
This change is not needed. If it does not progress within 5 seconds, then
it probably will not progress after another 25 seconds either. (It is
unrelated to the hang.)
---
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.
---
