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

2015-08-25 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #46 was successful. --- Scheduled 1062 tests in total. https://build.spring.io/browse/SGF-NAG-46/ -- This me

Review Request 37781: GEODE-139 Fixing RebalanceOperationDUnitTest.testWaitFor* methods

2015-08-25 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37781/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 37777: GEODE-233: Don't log a warning about a RegionDestroyedException

2015-08-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3/#review96445 --- Ship it! Ship It! - Darrel Schneider On Aug. 25, 2015, 4:03 p.m.

Review Request 37777: GEODE-233: Don't log a warning about a RegionDestroyedException

2015-08-25 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 37737: GEODE-275: fix race in testRegionExpirationAfterMutate

2015-08-25 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37737/#review96435 --- Ship it! Ship It! - Kirk Lund On Aug. 24, 2015, 11:23 p.m., Darr

Re: GEODE-77 status

2015-08-25 Thread Roman Shaposhnik
On Tue, Aug 25, 2015 at 1:08 PM, Anthony Baker wrote: > The work Bruce mentioned is proceeding on the GEODE-77 feature branch. > On that branch the gemfire-jgroups project containing the LGPL source has > been removed from the build (settings.gradle) but is still present in the > directory structu

Re: Review Request 37765: Fix for GEODE-278

2015-08-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37765/#review96415 --- Ship it! Since you have changed the behavior instead of it being co

Review Request 37769: Fixing failure from RegionDirectoryJUnitTest.testCopyBytesWithThreads

2015-08-25 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37769/ --- Review request for geode, Ashvin A and xiaojian zhou. Repository: geode Descr

Re: GEODE-77 status

2015-08-25 Thread Anthony Baker
The work Bruce mentioned is proceeding on the GEODE-77 feature branch. On that branch the gemfire-jgroups project containing the LGPL source has been removed from the build (settings.gradle) but is still present in the directory structure. The jgroups source code would be removed prior to merg

Re: Review Request 37765: Fix for GEODE-278

2015-08-25 Thread Swapnil Bawaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37765/ --- (Updated Aug. 25, 2015, 8:01 p.m.) Review request for geode and Darrel Schneide

Re: Review Request 37768: GEODE-279: fix hang caused by unhandled exception

2015-08-25 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37768/#review96395 --- Ship it! Ship It! - Dan Smith On Aug. 25, 2015, 6:21 p.m., Darre

Re: GEODE-77 status

2015-08-25 Thread Roman Shaposhnik
Hi! this is great news. But just clarify: does this mean that all of LGPL code is now gone from Geode codebase? Thanks, Roman. On Fri, Aug 21, 2015 at 4:36 PM, Bruce Schuchardt wrote: > Hi Geode devs, > > The GEODE-77 project to upgrade to a newer version of JGroups is moving > quickly. > > We

Re: Review Request 37765: Fix for GEODE-278

2015-08-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37765/#review96388 --- gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/TXCo

Review Request 37768: GEODE-279: fix hang caused by unhandled exception

2015-08-25 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37768/ --- Review request for geode and Dan Smith. Bugs: GEODE-279 https://issues.apac

Review Request 37765: Fix for GEODE-278

2015-08-25 Thread Swapnil Bawaskar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/37765/ --- Review request for geode and Darrel Schneider. Repository: geode Description

Re: Review Request 37720: Lucene: GEODE-11-Add-a-basic-score-based-aggregator

2015-08-25 Thread Ashvin A
> On Aug. 24, 2015, 8:59 p.m., Dan Smith wrote: > > gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/mergers/TopDocsResultMerger.java, > > line 29 > > > > > > Is is possible to lose a document

Re: Review Request 37720: Lucene: GEODE-11-Add-a-basic-score-based-aggregator

2015-08-25 Thread Ashvin A
> On Aug. 24, 2015, 8:59 p.m., Dan Smith wrote: > > gemfire-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/mergers/TopDocsResultMerger.java, > > line 29 > > > > > > Is is possible to lose a document

Jenkins build is back to normal : Geode-nightly #195

2015-08-25 Thread Apache Jenkins Server
See