Build failed in Jenkins: Geode-nightly #471

See Changes: [bschuchardt] GEODE-1393 locator returns incorrect server information when starting up [dschneider] GEODE-1252: modify bits field atomically [jiliao] GEODE-17: wording change [dschneider] GEODE-1222: remove InsufficientDis

Re: Review Request 47429: change defragment to not create fragments > 2G

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47429/#review133559 --- geode-core/src/main/java/com/gemstone/gemfire/internal/offheap/Fr

Re: Review Request 47433: handle NOT_AVAILABLE in callers of getRawOldValue

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47433/#review133566 --- Ship it! Ship It! - Sai Boorlagadda On May 17, 2016, 12:01 a

Re: Review Request 47433: handle NOT_AVAILABLE in callers of getRawOldValue

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47433/#review133570 --- Can we add a unit tests for this? - Ken Howe On May 17, 2016, 1

[GitHub] incubator-geode pull request: GEODE-1256 Alter rat.gradle to exclu...

Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/144#issuecomment-219785289 Should this stuff be added to the .gitignore as well? --- If your project is set up for it, you can reply to this email and have your reply appear on Gi

[GitHub] incubator-geode pull request: GEODE-1256 Alter rat.gradle to exclu...

Github user davebarnes97 commented on the pull request: https://github.com/apache/incubator-geode/pull/144#issuecomment-219786593 +1 - applied patch & ran a build, license errors are gone. The geode-site/content directory is already included in geode-site/.gitignore. --- If your

Re: Review Request 47433: handle NOT_AVAILABLE in callers of getRawOldValue

> On May 17, 2016, 10:03 a.m., Ken Howe wrote: > > Can we add a unit tests for this? I unit test could be added. I didn't think it was needed since it was a simple change. - Darrel --- This is an automatically generated e-mail. To repl

Re: Review Request 47433: handle NOT_AVAILABLE in callers of getRawOldValue

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47433/#review133577 --- Ship it! Yes, it's a simple change, but EntryEventImpl doesn't

Review Request 47479: GEODE-1296: change conditional in getRawBytes to assert

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47479/ --- Review request for geode, Darrel Schneider, Eric Shu, Scott Jewell, and Sai Boor

[GitHub] incubator-geode pull request: GEODE-1256 Alter rat.gradle to exclu...

Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/144 --- 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 featur

Re: Review Request 47479: GEODE-1296: change conditional in getRawBytes to assert

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47479/#review133580 --- Ship it! Ship It! - Darrel Schneider On May 17, 2016, 10:22

[GitHub] incubator-geode pull request: GEODE-1296: change conditional in ge...

GitHub user pdxrunner opened a pull request: https://github.com/apache/incubator-geode/pull/145 GEODE-1296: change conditional in getRawBytes to assert Made suggested change to getRawBytes. Updated corresponding unit test. See review request https://reviews.apache.org/r/47479/ Y

Posted [Video] Special Geode Clubhouse: Southwest Airlines' Experience with Geode & Gemfire at Scale

Dear Geode Community, I'm super happy to report we have posted the video of Brian Dunlap's second talk about his work with the Geode / Gemfire technology at Southwest Airlines. He goes in deep about his design patterns, and useful tips for deploying and operating Geode at the scale of Southwest Ai

Geode Portfile for Mac Ports

Using the geode.rb file for Homebrew as a template, I was able to build a Portfile for macports.org. Does anyone have any feedback before I submit it to the macports repo? No code changes were necessary and it is using the same binary image the Homebrew version is. I've tested it as far as "Geod

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #312 was SUCCESSFUL (with 1396 tests). Change made by John Blum .

--- Spring Data GemFire > Nightly-ApacheGeode > #312 was successful. --- Scheduled with changes by John Blum . 1398 tests in total. https://build.spring.io/brows

-Dtest.single= geode-core:test no longer works

Specifying -Dtest.single now always fails no matter which test (UnitTest) is specified. The integrationTest and distributedTest tasks still work with .single. Only test (UnitTest) is effected. It fails in :buildSrc:test despite specifying geode-core:test so I'm guessing it was the addition of buil

Re: Geode Portfile for Mac Ports

Awesome! -Dan On Tue, May 17, 2016 at 1:23 PM, Kevin Duling wrote: > Using the geode.rb file for Homebrew as a template, I was able to build a > Portfile for macports.org. > > Does anyone have any feedback before I submit it to the macports repo? No > code changes were necessary and it is usin

Re: -Dtest.single= geode-core:test no longer works

I thought the correct syntax was more like below, where you prefix the target with geode-core, or geode-lucene, etc. ./gradlew geode-core:test -Dtest.single=StringUtilsJUnitTest For me, this works. If I leave off the geode-core: part I get a failure from every other subproject (geode-core, geode-

Re: Geode Portfile for Mac Ports

nice! On Tue, May 17, 2016 at 2:05 PM, Dan Smith wrote: > Awesome! > > -Dan > > On Tue, May 17, 2016 at 1:23 PM, Kevin Duling wrote: > > > Using the geode.rb file for Homebrew as a template, I was able to build a > > Portfile for macports.org. > > > > Does anyone have any feedback before I subm

Review Request 47494: GEODE-1378: Don't launch dunit VMs in checkMissedTests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47494/ --- Review request for geode, Anthony Baker and Bruce Schuchardt. Repository: geode

Re: -Dtest.single= geode-core:test no longer works

It used to work both ways. The integrationTest and distributedTest targets still work both ways. I've only ever used the syntax with the ordering of -D before module:target because that's what is documented on the gradle website: 45.13.4. Single test execution via System Properties https://docs.gr

Review Request 47496: Adding transaction tests for lucene indexes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47496/ --- Review request for geode, anilkumar gingade, Barry Oglesby, nabarun nag, Dan Smi

Re: Review Request 47494: GEODE-1378: Don't launch dunit VMs in checkMissedTests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47494/#review133654 --- Ship it! The lock service test changes look fine to me - Bruce

Re: Review Request 47429: change defragment to not create fragments > 2G

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47429/ --- (Updated May 17, 2016, 4:13 p.m.) Review request for geode, Eric Shu, Scott Jew

Review Request 47501: GEODE-1387 CI Failure: LocatorLauncherRemoteFileIntegrationTest.testStatusUsingWorkingDirectory

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47501/ --- Review request for geode, Bruce Schuchardt, Hitesh Khamesra, Kirk Lund, and Udo

Review Request 47503: GEODE-357: Remove third server from testInitiateFailoverByCacheOperationThreads_Secondary

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47503/ --- Review request for geode and xiaojian zhou. Repository: geode Description ---

Review Request 47504: GEODE-1133 SeparateClassloaderTestRunner has to be re-implemented

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47504/ --- Review request for geode, Bruce Schuchardt, Hitesh Khamesra, Jens Deppe, and Udo

Re: Review Request 47496: Adding transaction tests for lucene indexes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47496/#review133663 --- Fix it, then Ship it! Ship It! geode-lucene/src/test/java/com

Re: Review Request 47501: GEODE-1387 CI Failure: LocatorLauncherRemoteFileIntegrationTest.testStatusUsingWorkingDirectory

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47501/#review133665 --- Ship it! Ship It! - Kirk Lund On May 17, 2016, 11:49 p.m., J

Re: Geode Portfile for Mac Ports

Awesome work, but please consider calling it apache-geode. Thanks, Roman. On Tue, May 17, 2016 at 1:23 PM, Kevin Duling wrote: > Using the geode.rb file for Homebrew as a template, I was able to build a > Portfile for macports.org. > > Does anyone have any feedback before I submit it to the macp

Re: Review Request 47416: GEODE-17: make geode authorization case-sensitive since our region names are case sensitive

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47416/#review133667 --- Fix it, then Ship it! Fix it and Ship It! Two minor style chan

Re: Review Request 47372: GEODE-1209: Added gfsh support to set ignoreEvictionAndExpiration attribute for create AEQ command.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47372/#review133669 --- Ship it! Ship It! - Kirk Lund On May 13, 2016, 11:05 p.m., a

Re: Review Request 47429: change defragment to not create fragments > 2G

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47429/#review133673 --- Fix it, then Ship it! Ship It! geode-core/src/main/java/com/g

Re: Review Request 47496: Adding transaction tests for lucene indexes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47496/#review133671 --- Fix it, then Ship it! Looks good! Let's talk tomorrow a little

Re: Review Request 47416: GEODE-17: make geode authorization case-sensitive since our region names are case sensitive

> On May 18, 2016, 12:38 a.m., Kirk Lund wrote: > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/DistributionConfig.java, > > line 61 > > > > > > Geode style is just to have this all on one line

Re: Review Request 47503: GEODE-357: Remove third server from testInitiateFailoverByCacheOperationThreads_Secondary

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47503/#review133694 --- Ship it! Ship It! - xiaojian zhou On May 17, 2016, 11:58 p.m

Re: Geode Portfile for Mac Ports

+1 Cool stuff! With brew and ports now we should probably add those to the wiki as alternatives to install Geode. Next steps could be rpm or deb packages. ;) Sent from my iPhone > On May 17, 2016, at 7:36 PM, Roman Shaposhnik wrote: > > Awesome work, but please consider calling it apache