Re: What will it take to release geode 1.0?

2016-05-03 Thread William Markito
Great discussion... On Tue, May 3, 2016 at 6:45 PM, Brian Dunlap wrote: > R1 without package renaming sounds good. > Cool. > > Along with the R1 release info, it would be uber-helpful to clarify > cheese-moving public (and internal) API munging that's cooking with R2. >

proposal for pluggable stores

2016-05-03 Thread Shankar Hundekar
Hi All, Please review the project proposal for the geode pluggable stores posted on the geode wiki at https://cwiki.apache.org/confluence/display /GEODE/Pluggable+Cache+Persistence+and+Overflow+Stores Please post your review comments on the wiki page. Thanks, Shankar

Re: What will it take to release geode 1.0?

2016-05-03 Thread Brian Dunlap
R1 without package renaming sounds good. Along with the R1 release info, it would be uber-helpful to clarify cheese-moving public (and internal) API munging that's cooking with R2.

Re: New Summary & Proceedings of Geode Summit Posted!

2016-05-03 Thread Gregory Chase
Dear Community, I have to apologize. Several hours of work adding links to videos and slides didn't get saved for some reason. You can also find everything here: https://2016.event.geodesummit.com/schedule I'll try to fix this later. -Greg On Tue, May 3, 2016 at 5:41 PM, Gregory Chase

Re: Review Request 46926: GEODE-1260: Cache SCM metadata for source distributions

2016-05-03 Thread Anthony Baker
> On May 4, 2016, 12:42 a.m., Dick Cavender wrote: > > I assume we can still dump the GemFireVersion.properties by running gfsh > > version --full? Sure. In fact, with this change someone who build geode from the source distribution will get the actual version instead of UNKNOWN. Shouldn't

Re: Review Request 46243: GEODE-1209: Added new attribute to forward eviction/expiration to AEQ. Tested with manual testing.

2016-05-03 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46243/ --- (Updated May 4, 2016, 1:09 a.m.) Review request for geode, anilkumar gingade,

Re: Review Request 46926: GEODE-1260: Cache SCM metadata for source distributions

2016-05-03 Thread Anthony Baker
> On May 4, 2016, 12:30 a.m., Dan Smith wrote: > > geode-core/build.gradle, line 155 > > > > > > Should we catch a more specific exception here? Otherwise if this > > readSCMInfo code breaks we might never know.

Re: Review Request 46926: GEODE-1260: Cache SCM metadata for source distributions

2016-05-03 Thread Dick Cavender
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46926/#review131599 --- Ship it! I assume we can still dump the

New Summary & Proceedings of Geode Summit Posted!

2016-05-03 Thread Gregory Chase
Dear Geode Community, I just published a new version of the Geode Summit proceedings with links to all the session slides and replays. If you want to revisit, or virtually attend last March's Geode Summit, please feel free to view the 22 deep dive sessions in all their amazing detail:

Re: What will it take to release geode 1.0?

2016-05-03 Thread Anthony Baker
Seems fine. I just double-checked and jackson v2.2.0 does seem to be still available on maven central so perhaps GEODE-612 can be closed entirely. Anthony > On May 3, 2016, at 5:07 PM, William Markito wrote: > > @Anthony, what if we did: > > # M3 > GEODE-1028

Re: Review Request 46926: GEODE-1260: Cache SCM metadata for source distributions

2016-05-03 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46926/#review131595 --- Fix it, then Ship it! geode-core/build.gradle (line 147)

Re: What will it take to release geode 1.0?

2016-05-03 Thread William Markito
@Anthony, what if we did: # M3 GEODE-1028 Broken website link GEODE-1133 SeparateClassloaderTestRunner GEODE-1260 Source distribution version info # 1.0.0 GEODE-1191 HDFS references GEODE-612Update Jackson version since current version is not on Maven central Reasoning here being that

Javadocs warnings and errors

2016-05-03 Thread Kirk Lund
Lots of new warnings and errors are being introduced. Please watch what you're checking in. This is not valid javadocs: /** * * @return */

Re: Review Request 46918: GEODE-11: Refactoring the LuceneFunctionReadPathDUnitTest

2016-05-03 Thread Jason Huynh
> On May 3, 2016, 9:20 p.m., Jason Huynh wrote: > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesBase.java, > > line 60 > > > > > > Should we pull the "createIndex" into it's own method

Re: What will it take to release geode 1.0?

2016-05-03 Thread bdunlap22
Did Kirk's note about package renaming make the M3 scope? I think it would be great to start Geode's 1.0 on a consistent naming standard. :) Pro-Geode!!! Brian - Sent from my iPhone > On May 3, 2016, at 6:09 PM, Anthony Baker wrote: > > William, > > What do you think

Re: Review Request 46918: GEODE-11: Refactoring the LuceneFunctionReadPathDUnitTest

2016-05-03 Thread Dan Smith
> On May 3, 2016, 9:20 p.m., Jason Huynh wrote: > > geode-lucene/src/test/java/com/gemstone/gemfire/cache/lucene/LuceneQueriesBase.java, > > line 60 > > > > > > Should we pull the "createIndex" into it's own method

Re: What will it take to release geode 1.0?

2016-05-03 Thread Anthony Baker
William, What do you think about including these in M3? GEODE-612 Update Jackson version since current version is not on Maven central GEODE-1028 Broken website link GEODE-1191 HDFS references GEODE-1133 SeparateClassloaderTestRunner GEODE-1260 Source distribution version info

Re: What will it take to release geode 1.0?

2016-05-03 Thread William Markito
Guys, restarting this thread to get a discussion going about M3, 1.0.0 and next - As the release manager for M3 here is what I'd like to propose. Any feedback is welcome and let's also reuse this thread to talk a little bit about roadmap as well ? # Current M3 Scope (

Re: Do we want a "theme song" for Geode Cluhouse?

2016-05-03 Thread james bedenbaugh
Meh. Talk to Stoltz. He's got budget. . .  On Tue, May 3, 2016 at 5:16 PM, Gregory Chase wrote: > On Tue, May 3, 2016 at 3:11 PM, james bedenbaugh > wrote: > > > Well, since we are nominating, let's use the Wayback Machine. . .and in > no > >

Re: Do we want a "theme song" for Geode Cluhouse?

2016-05-03 Thread Gregory Chase
On Tue, May 3, 2016 at 3:11 PM, james bedenbaugh wrote: > Well, since we are nominating, let's use the Wayback Machine. . .and in no > particular order: > > >- Born to Wild - >Steppenwolf >- Brown Sugar

Re: Do we want a "theme song" for Geode Cluhouse?

2016-05-03 Thread Michael Stolz
Yes I do. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: 631-835-4771 On Tue, May 3, 2016 at 6:02 PM, Gregory Chase wrote: > +1 Love it! Do you give permission to Apache Geode community contributors > to make use of your original work as part of Apache

Re: Review Request 46918: GEODE-11: Refactoring the LuceneFunctionReadPathDUnitTest

2016-05-03 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46918/#review131560 --- Fix it, then Ship it!

Re: Review Request 46917: GEODE-625: Adding tests for exception handling of functions

2016-05-03 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46917/#review131559 --- Ship it! Ship It! - Jason Huynh On May 3, 2016, 1:03 a.m.,

Re: Review Request 46940: GEODE-1329 auto-reconnect attempts cease if kicked out during boot-up of the cache

2016-05-03 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46940/#review131540 --- Ship it! Ship It! - Udo Kohlmeyer On May 3, 2016, 3:43

Re: Review Request 46940: GEODE-1329 auto-reconnect attempts cease if kicked out during boot-up of the cache

2016-05-03 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46940/#review131537 --- Ship it! Ship It! - Jianxia Chen On May 3, 2016, 3:43 p.m.,

[GitHub] incubator-geode pull request: GEODE-1121: Increased the max memory...

2016-05-03 Thread upthewaterspout
Github user upthewaterspout commented on the pull request: https://github.com/apache/incubator-geode/pull/139#issuecomment-216614483 +1 looks good, but maybe we should scale this test way down. Why does it need to put this much data in order to test this feature? Could we bring the

GMSHealthMonitor Sequence Diagrams

2016-05-03 Thread Jianxia Chen
I have added sequence diagrams for GMSHealthMonitor that monitors the health of Geode distributed system. Have a look: https://cwiki.apache.org/confluence/display/GEODE/GMSHealthMonitor+Message+Sequence+Diagram

Re: Review Request 46896: GEODE-1327 GMSJoinLeave: got java.util.ConcurrentModificationException while updating log message

2016-05-03 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46896/#review131528 --- I like this a lot better

Re: Review Request 46896: GEODE-1327 GMSJoinLeave: got java.util.ConcurrentModificationException while updating log message

2016-05-03 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46896/ --- (Updated May 3, 2016, 4:46 p.m.) Review request for geode, Bruce Schuchardt,

Review Request 46940: GEODE-1329 auto-reconnect attempts cease if kicked out during boot-up of the cache

2016-05-03 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46940/ --- Review request for geode and Jianxia Chen. Bugs: GEODE-1329

Re: Review Request 46937: GEODE-17 - use null instead of "NULL" for regionName

2016-05-03 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46937/#review131511 --- Ship it! Ship It! - Jens Deppe On May 3, 2016, 2:57 p.m.,

Reminder: Geode Clubhouse TOMORROW Weds, May 4, 9AM PDT - Where Does Geode Fit in Modern System Architectures?

2016-05-03 Thread Gregory Chase
Greetings Apache Geode Community, This is a reminder that we're hosting a Geode Clubhouse meeting tomorrow. See you there! As always it will be recorded and posted. -Greg On Wed, Apr 27, 2016 at 2:44 PM, Gregory Chase wrote: > It's time for the next Apache Geode Clubhouse

Review Request 46937: GEODE-17 - use null instead of "NULL" for regionName

2016-05-03 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46937/ --- Review request for geode, Jens Deppe and Kirk Lund. Repository: geode

Re: Review Request 46850: GEODE-1329 auto-reconnect attempts cease if kicked out during boot-up of the cache

2016-05-03 Thread Bruce Schuchardt
> On May 2, 2016, 11:34 p.m., Jianxia Chen wrote: > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/InternalDistributedSystem.java, > > line 2748 > > > > > > Why this one does not need a lock?