Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/316#issuecomment-64132928
@javrasya @HeartSaVioR I don't understand this change. We already
discussed this in another JIRA that having systemProperty doesn't pass in the
variable to testing.clj. I tested this PR with following command
"mvn clean install -DstormTestTimeoutMs=20000"
and my test timeout still 5000.
If the intention of this patch is for other tests to increase the timeout
they can pass it as parameter to
(defn wait-until-cluster-waiting
"Wait until the cluster is idle. Should be used with time simulation."
([cluster-map] (wait-until-cluster-waiting cluster-map TEST-TIMEOUT-MS))
([cluster-map timeout-ms]
---
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.
---
