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

2016-08-03 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #390 was successful. --- Scheduled 1425 tests in total. https://build.spring.io/browse/SGF-NAG-390/ -- This

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Barry Oglesby
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50680/#review144660 --- Ship it! Ship It! - Barry Oglesby On Aug. 3, 2016, 6:36

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider
> On Aug. 3, 2016, 11:35 a.m., Darrel Schneider wrote: > > This fix has a problem. It always reads the value from disk if it had > > overflowed. > > But just a couple of lines before this basicPut has called > > setOldValueInEvent. > > I think the correct fix is to use that old value from the

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50680/ --- (Updated Aug. 3, 2016, 11:36 a.m.) Review request for geode, Barry Oglesby,

Re: Review Request 50680: fix replace on overflowed entry

2016-08-03 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50680/#review144640 --- This fix has a problem. It always reads the value from disk if it

[GitHub] incubator-geode pull request #226: GEODE-1712: introduce SecurityService int...

2016-08-03 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/226 GEODE-1712: introduce SecurityService interface for mocking GEODE-1712: add missing categories and fix test failure * fix changes recommended in review GEODE-1712: add static

[GitHub] incubator-geode pull request #223: GEODE-1712: introduce SecurityService and...

2016-08-03 Thread gracemeilen
Github user gracemeilen closed the pull request at: https://github.com/apache/incubator-geode/pull/223 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

Build failed in Jenkins: Geode-release #23

2016-08-03 Thread Apache Jenkins Server
See -- [...truncated 801 lines...] :geode-lucene:compileJavaNote: Some input files use or override a deprecated API. Note: Recompile with -Xlint:deprecation for details. Note: Some input files use unchecked

Re: Build failed in Jenkins: Geode-nightly #549

2016-08-03 Thread Bruce Schuchardt
The port-range test failure is due to my checkin yesterday. I'm looking into it. Le 8/3/2016 à 7:28 AM, Apache Jenkins Server a écrit : See Changes: [jdeppe] GEODE-1666: Bump Gradle from 2.12 to 2.14.1 [klund] GEODE-1682: Adding

Build failed in Jenkins: Geode-nightly #549

2016-08-03 Thread Apache Jenkins Server
See Changes: [jdeppe] GEODE-1666: Bump Gradle from 2.12 to 2.14.1 [klund] GEODE-1682: Adding options for starting Geode REST API using gfsh [klund] GEODE-1682: Added test for the REST arguments and Use correct [klund] GEODE-1682:

[GitHub] incubator-geode pull request #225: Fix e.printStackTrace() in ClientPartitio...

2016-08-03 Thread omallassi
GitHub user omallassi opened a pull request: https://github.com/apache/incubator-geode/pull/225 Fix e.printStackTrace() in ClientPartitionAdvisor the exception was logged using e.printstacktrace(), switch to use the LoggerService. You can merge this pull request into a Git