samza-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From shanthoosh <...@git.apache.org>
Subject [GitHub] samza pull request #108: SAMZA-1183: Fix async run loop tests
Date Mon, 03 Apr 2017 21:40:12 GMT
GitHub user shanthoosh opened a pull request:

    https://github.com/apache/samza/pull/108

    SAMZA-1183: Fix async run loop tests

    Changes
    * Moved all states locally within the test methods(no global state, since tests can be
run parallely).
    * Increased the timeout waiting for all task threads to finish.
    * Changed message chooser mock to return null at the end(When EOS message is used a final
msg, during some runs task.process gets invoked more than once for it. This is due to test
setup, other tests within this test class also do the same.)
    
    How was it verified: 
    
    Ran the test class repeatedly to check for failures(there were none).
    
    `
    while [ $i -lt 30 ] do     sleep 3;     i=`expr $i + 1`;     ./gradlew :samza-core:clean
:samza-core:test -Dtest.single=TestAsyncRunLoop  done
    `
    
    


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

    $ git pull https://github.com/shanthoosh/samza fix-async-run-loop-tests-1

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

    https://github.com/apache/samza/pull/108.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 #108
    
----
commit 2505a9d5ddff3bed441319d9a122c25cd4e8f2f8
Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com>
Date:   2017-04-03T21:11:10Z

    SAMZA-1183: Fixing async run loop non blocking commit tests.
    
    Changes
    * Moved all states within the tests(no global state, since tests can be run parallely).
    * Increased the timeout waiting for all task threads to finish.
    * Changed message chooser mock to return null at the end(Sometimes when EOS message is
used a final msg,
    task.process gets invoked more than once for it.)

----


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message