GitHub user d2r opened a pull request:

    https://github.com/apache/storm/pull/966

    [STORM-1401] removes multilang-test

    This test takes over 2 minutes to run, and it does not validate correctness 
of the multilang code.  It basically runs the topologies for a while and passes 
if they do not crash. Since they are now causing repeated failures in automated 
testing, there is not a lot of value in keeping them as-is.
    
    We should cover multilang code as well as the Shell* classes in unit 
testing.  So I will create a follow-on issue to cover these with tests.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/d2r/storm storm-1401-multilang-test-removal

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/966.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #966
    
----
commit 37099d8ee68cd8c440bae34ed2aa0c7f30a4d3c2
Author: Derek Dagit <[email protected]>
Date:   2015-12-22T20:46:37Z

    removes multilang-test

----


---
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.
---

Reply via email to