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

Posted to dev@geode.apache.org by vivek bhaskar <vi...@apache.org> on 2015/08/03 14:19:44 UTC

Re: Review Request 36936: GEODE-172 DistTXPersistentDebugDUnitTest.testTXPRRR2_putall fail intermittently because cache already exists

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36936/
-----------------------------------------------------------

(Updated Aug. 3, 2015, 12:19 p.m.)


Review request for geode, Darrel Schneider and Shirish Deshmukh.


Changes
-------

Additional changes to close cache on all VMs and clear system property while closing tests.


Bugs: GEODE-172
    https://issues.apache.org/jira/browse/GEODE-172


Repository: geode


Description
-------

Fixing an intermittent failure as reported by closing cache after each individual test.

Also taking care to avoid an assertion failure that is reflected in GEODE-110 and GEODE-141.


Diffs (updated)
-----

  gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistTXDebugDUnitTest.java 668f1e6 
  gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistributedTransactionDUnitTest.java 79d6f64 

Diff: https://reviews.apache.org/r/36936/diff/


Testing
-------

./gradlew gemfire-core:integrationTest -DintegrationTest.single=*DistTX*JUnit*
./gradlew -DdistributedTest.single=DistTXDistributedTestSuite gemfire-core:distributedTest


Thanks,

vivek bhaskar