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 <[email protected]>
Date: 2017-04-04T21:58:15Z
Fixing testLatchSizeOne
commit 2675d7e1be0eb21849382e456ae2099b53dd02e5
Author: navina <[email protected]>
Date: 2017-04-04T22:18:13Z
Fixed testLatchSizeOneWithTwoParticipants and remove ZkConfig from
ZkProcessorLatch constructor
commit 923cbb3aec085f39ffe532b49c2ef02f428b6b09
Author: navina <[email protected]>
Date: 2017-04-04T23:14:50Z
Fixed testLatchSizeN
commit efdc14a55f99deea7061ce97adac919a82c7f3e8
Author: navina <[email protected]>
Date: 2017-04-04T23:19:30Z
Changing certain member variables to local variables
commit fc7b23378d32d59e2890522d141505fec66d4179
Author: navina <[email protected]>
Date: 2017-04-04T23:24:25Z
Renaming processorId member variable to participantId
commit 76e6bae0d506f793ea38b5d159a1d48596e98890
Author: navina <[email protected]>
Date: 2017-04-04T23:46:33Z
Fixing testLatchExpires
commit ed62f9d6e764c9f3655ef4b9c147496ab4fca96c
Author: navina <[email protected]>
Date: 2017-04-04T23:49:15Z
Changing sop to log statements
commit 0631d9bc273146fec9c6fb3df897e27a50a51a17
Author: navina <[email protected]>
Date: 2017-04-05T03:09:48Z
Fixing checkstyle
commit 7d39906a963168a67059f822192e2dd15ccfad45
Author: navina <[email protected]>
Date: 2017-04-05T03:19:02Z
Refactoring runnable code
commit 82afa27a62ce5742a1796dd1fb23977b014fefb6
Author: navina <[email protected]>
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 [email protected] or file a JIRA ticket
with INFRA.
---
