GitHub user navina opened a pull request:

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

    Samza-1187 : TestZkProcessorLatch tests share state causing transient 
failures in CI builds

    * Removed testSingleCountdown - didn't understand what was being tested
    * The timeout behavior for ZkProcessorLatch has not been documented. Hence, 
I have fixed testLatchExpires as best I could understand. 
    * Removed redundant/unused member variables from ZkProcessorLatch

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

    $ git pull https://github.com/navina/samza SAMZA-1187

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

    https://github.com/apache/samza/pull/112.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 #112
    
----
commit 2d2b167b63d91793988f63ac6be44eebf9294c20
Author: navina <nav...@apache.org>
Date:   2017-04-04T21:58:15Z

    Fixing testLatchSizeOne

commit 2675d7e1be0eb21849382e456ae2099b53dd02e5
Author: navina <nav...@apache.org>
Date:   2017-04-04T22:18:13Z

    Fixed testLatchSizeOneWithTwoParticipants and remove ZkConfig from 
ZkProcessorLatch constructor

commit 923cbb3aec085f39ffe532b49c2ef02f428b6b09
Author: navina <nav...@apache.org>
Date:   2017-04-04T23:14:50Z

    Fixed testLatchSizeN

commit efdc14a55f99deea7061ce97adac919a82c7f3e8
Author: navina <nav...@apache.org>
Date:   2017-04-04T23:19:30Z

    Changing certain member variables to local variables

commit fc7b23378d32d59e2890522d141505fec66d4179
Author: navina <nav...@apache.org>
Date:   2017-04-04T23:24:25Z

    Renaming processorId member variable to participantId

commit 76e6bae0d506f793ea38b5d159a1d48596e98890
Author: navina <nav...@apache.org>
Date:   2017-04-04T23:46:33Z

    Fixing testLatchExpires

commit ed62f9d6e764c9f3655ef4b9c147496ab4fca96c
Author: navina <nav...@apache.org>
Date:   2017-04-04T23:49:15Z

    Changing sop to log statements

commit 0631d9bc273146fec9c6fb3df897e27a50a51a17
Author: navina <nav...@apache.org>
Date:   2017-04-05T03:09:48Z

    Fixing checkstyle

commit 7d39906a963168a67059f822192e2dd15ccfad45
Author: navina <nav...@apache.org>
Date:   2017-04-05T03:19:02Z

    Refactoring runnable code

commit 82afa27a62ce5742a1796dd1fb23977b014fefb6
Author: navina <nav...@apache.org>
Date:   2017-04-05T06:56:29Z

    adjusting spacing

----


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

Reply via email to