You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "kirklund (GitHub)" <gi...@apache.org> on 2019/11/11 17:26:23 UTC

[GitHub] [geode] kirklund opened pull request #4308: GEODE-6818: Fix flakiness in PutAllClientServerDistributedTest

GEODE-6818: Fix flakiness in PutAllClientServerDistributedTest
    
Overhaul test and fix sources of flakiness (wait/notify and sleeps).
    
* Replace object wait/notify with CountDownLatch(es)
* Replace sleeps with Awaitility
* Inline super-class and lots of methods
* Remove unused or unnecessary code
* Update deprecated API usage
* Update to AssertJ and Awaitility

[ Full content available at: https://github.com/apache/geode/pull/4308 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode] mhansonp commented on pull request #4308: GEODE-6818: Fix flakiness in PutAllClientServerDistributedTest

Posted by "mhansonp (GitHub)" <gi...@apache.org>.

Can you make this test name more descriptive of what actually happens or add a comment? 

[ Full content available at: https://github.com/apache/geode/pull/4308 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org