Re: Review Request 36594: Adding try-with-resources for Cache

2015-07-22 Thread William Markito
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36594/ --- (Updated July 23, 2015, 6:19 a.m.) Review request for geode, Darrel Schneider a

Re: Review Request 36594: Adding try-with-resources for Cache

2015-07-22 Thread William Markito
> On July 20, 2015, 5:25 p.m., Darrel Schneider wrote: > > gemfire-core/src/test/java/com/gemstone/gemfire/cache/RegionFactoryJUnitTest.java, > > line 1113 > > > > > > I'm not sure this unit test needs to create a

Re: Review Request 36632: [#99282642] (Merging the changes done in Changeset 44816]

2015-07-22 Thread Namrata Thanvi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36632/ --- (Updated July 23, 2015, 5:36 a.m.) Review request for geode and Swapnil Bawaska

Build failed in Jenkins: Geode-nightly #164

2015-07-22 Thread Apache Jenkins Server
See Changes: [dsmith] GEODE-56 Decrementing bytesOverflowedOnDisk when an update happens to an evicted entry. [dsmith] GEODE-149 Reexecute a test even it previously passed -- [...truncated 151 lin

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #12 was SUCCESSFUL (with 965 tests)

2015-07-22 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #12 was successful. --- Scheduled 969 tests in total. https://build.spring.io/browse/SGF-NAG-12/ -- This mes

Review Request 36714: GEODE-143: logging must not cause cached values to be deserialized

2015-07-22 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36714/ --- Review request for geode and Kirk Lund. Repository: geode Description ---

Review Request 36695: GEODE-148: change expiration tests to not run so long

2015-07-22 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36695/ --- Review request for geode and Kirk Lund. Repository: geode Description ---

Re: Review Request 36694: Re-introduce 4 dunit tests that were exclude earlier.

2015-07-22 Thread Ashvin A
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36694/#review92601 --- Ship it! Ship It! - Ashvin A On July 22, 2015, 4:02 p.m., Amogh

Review Request 36694: Re-introduce 4 dunit tests that were exclude earlier.

2015-07-22 Thread Amogh Shetkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36694/ --- Review request for geode and Ashvin A. Bugs: GEODE-119 https://issues.apach

Re: Running the same unit test multiple times with gradle

2015-07-22 Thread vivek bhaskar
Thanks Dan. This helped. Regards, Vivek On Wed, Jul 22, 2015 at 12:49 PM, Amogh Shetkar wrote: > Thanks Dan. > Thanks Kirk, I too suspected test-pollution to be the case here and hence > wanted to run the same suite of tests together multiple times to repro the > issue. > > On Wed, Jul 22, 2015

Re: Running the same unit test multiple times with gradle

2015-07-22 Thread Amogh Shetkar
Thanks Dan. Thanks Kirk, I too suspected test-pollution to be the case here and hence wanted to run the same suite of tests together multiple times to repro the issue. On Wed, Jul 22, 2015 at 6:24 AM, Kirk Lund wrote: > Rerunning a test repeatedly may not reproduce some failures that Jenkins > m