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

2015-08-03 Thread vivek bhaskar
> On July 31, 2015, 4:47 p.m., Darrel Schneider wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistTXDebugDUnitTest.java, > > line 65 > > > > > > It is not clear to me why: > > 1. you are closi

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

2015-08-03 Thread vivek bhaskar
> On July 31, 2015, 4:35 p.m., Darrel Schneider wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistributedTransactionDUnitTest.java, > > line 75 > > > > > > I know you are commenting this out but m

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

2015-08-03 Thread vivek bhaskar
--- 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 a

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

2015-07-31 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/#review93739 --- gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistTXDebugD

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

2015-07-31 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/#review93737 --- gemfire-core/src/test/java/com/gemstone/gemfire/disttx/DistributedT

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

2015-07-30 Thread vivek bhaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/ --- (Updated July 31, 2015, 1:54 a.m.) Review request for geode and Shirish Deshmuk

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

2015-07-30 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/#review93593 --- Ship it! I think you accidently set the "bug" and "branch" to 142 i

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

2015-07-30 Thread Shirish Deshmukh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/#review93573 --- Ship it! Ship It! - Shirish Deshmukh On July 30, 2015, 10:33 a.m

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

2015-07-30 Thread vivek bhaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36936/ --- Review request for geode and Shirish Deshmukh. Bugs: GEODE-142 https://issu