GEODE 1.2 and java 8

2017-09-07 Thread Roi Apelker
Hi, As we have moved to Java 8, and compiled GEODE 1.2 with it, Are there any recommendations regarding java parameters (JVM or any other) that can be suggested? Thanks Roi This message and the information contained herein is proprietary and confidential and subject to the Amdocs policy

Build failed in Jenkins: Geode-nightly-flaky #115

2017-09-07 Thread Apache Jenkins Server
See -- Started by upstream project "Geode-nightly" build number 947 originally caused by: Started by timer [EnvInject] - Loading node environment variables. Building remotely on H26

Build failed in Jenkins: Geode-nightly #947

2017-09-07 Thread Apache Jenkins Server
See Changes: [jdeppe] GEODE-3556: Update gradle docker plugin to 0.5.4 [jdeppe] GEODE-3556: Add support for setting the user inside docker containers [boglesby] GEODE-3316: Modified tests to roll locator and

Query mechanism

2017-09-07 Thread Roi Apelker
Hi, Is there a design document which explains the query mechanism? Main classes, modules, main algorithm? I have been looking into this area (since I have a performance issue, where it seems that data is loaded from disk prematurely in eviction state, which causes the system to become really

Re: Missing Gitbox activation email

2017-09-07 Thread Nabarun Nag
Thanks Jared, that worked for me. On Thu, Sep 7, 2017 at 10:28 AM Jared Stewart wrote: > To anyone else having the same problem, I was able to accept the invite by > navigating to https://github.com/apache . > > - Jared > > On Sep 7, 2017, at

Re: Missing Gitbox activation email

2017-09-07 Thread Udo Kohlmeyer
It was as Dan indicated you need to make sure your github username is registered with id.apache.org. I then later received an email that requested I join the apache org. And now my gitbox works. Thank you for the help. --Udo On 9/6/17 21:41, Jacob Barrett wrote: I think the key is

Re: Review Request 62088: GEODE-3249 Validate internal client/server messages

2017-09-07 Thread Bruce Schuchardt
> On Sept. 5, 2017, 11 a.m., Hitesh Khamesra wrote: > > Ship It! I've had to do more work on this & would appreciate another review. - Bruce --- This is an automatically generated e-mail. To reply, visit:

Re: Review Request 62163: GEODE-3096: pulling in refactoring work on HttpOperationInvoker

2017-09-07 Thread Patrick Rhomberg
> On Sept. 7, 2017, 6:03 p.m., Patrick Rhomberg wrote: > > Overall: I love how much is getting pruned! > > > > I like to run inspections with the `Changed Files` scope to clean up some > > of the low-hanging fruit. In particular, I get rankled by > > * Missorted modifiers

Re: Review Request 62164: GEODE-3566 Moving a bucket during rebalancing does not update overflow stats

2017-09-07 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62164/#review184863 --- Ship it! Ship It! - anilkumar gingade On Sept. 7, 2017,

Re: Query mechanism

2017-09-07 Thread Anilkumar Gingade
Roi, There is no documentation on the internals of Query engine...I believe there are tools which can generate package/class diagrams and their relationship...Based on the problem/issue, we could point to the source file, where you can find more info. To know how query is picking the best index;

Re: Review Request 62164: GEODE-3566 Moving a bucket during rebalancing does not update overflow stats

2017-09-07 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62164/#review184872 ---

Re: Review Request 62132: GEODE-3277: Fix error path constructors of Launcher inner State classess

2017-09-07 Thread Ken Howe
> On Sept. 7, 2017, 6:16 p.m., Jared Stewart wrote: > > geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java > > Line 1099 (original), 1092 (patched) > > > > > > Do you think there is any

Re: Review Request 62132: GEODE-3277: Fix error path constructors of Launcher inner State classess

2017-09-07 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62132/#review184852 ---

Re: Review Request 62164: GEODE-3566 Moving a bucket during rebalancing does not update overflow stats

2017-09-07 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62164/#review184842 ---

Re: Review Request 62088: GEODE-3249 Validate internal client/server messages

2017-09-07 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62088/ --- (Updated Sept. 7, 2017, 10:43 a.m.) Review request for geode, Alexander

Review Request 62163: GEODE-3096: pulling in refactoring work on HttpOperationInvoker

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62163/ --- Review request for geode, Jared Stewart, Ken Howe, Kirk Lund, and Patrick

Review Request 62164: GEODE-3566 Moving a bucket during rebalancing does not update overflow stats

2017-09-07 Thread Lynn Gallinat
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62164/ --- Review request for geode, anilkumar gingade, Darrel Schneider, Eric Shu, and

Re: Monitor the neighbour JVM using neihbour's member-timeout

2017-09-07 Thread Bruce Schuchardt
I think this might be an acceptable change though I doubt many people would find it useful. It's already possible to set different member-timeouts on each node of the distributed system but the meaning of the setting is the inverse of what's proposed here, so having the current setting be

Re: Missing Gitbox activation email

2017-09-07 Thread Jared Stewart
I never received an Apache org invite on GitHub or via email after adding my GitHub username to id.apache.org. I filed an infra ticket to request help. (https://issues.apache.org/jira/browse/INFRA-15042 ) - Jared > On Sep 7, 2017, at 9:36

Re: Missing Gitbox activation email

2017-09-07 Thread Jared Stewart
To anyone else having the same problem, I was able to accept the invite by navigating to https://github.com/apache . - Jared > On Sep 7, 2017, at 9:36 AM, Udo Kohlmeyer wrote: > > It was as Dan indicated you need to make sure your github

Geode sessions at SpringOne Platform conference

2017-09-07 Thread Jagdish Mirani
Hello Geode contributors: As you most likely already know, we're folding the next Geode Summit into the upcoming SpringOne Platform conference. We're working through some great session proposals, and we'll keep adding the accepted sessions to the Geode page on the SpringOne Platform site.

Re: Review Request 62163: GEODE-3096: pulling in refactoring work on HttpOperationInvoker

2017-09-07 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62163/#review184843 --- Overall: I love how much is getting pruned! I like to run

Re: Review Request 62088: GEODE-3249 Validate internal client/server messages

2017-09-07 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62088/ --- (Updated Sept. 7, 2017, 10:32 a.m.) Review request for geode, Alexander

Re: [Discuss] Enable region set operations to start JTA transactions

2017-09-07 Thread Swapnil Bawaskar
How is this going to work for a PartitionedRegion? Are we going to throw TransactionDataNotColocatedException when iterating over non-colocated data in the PR? If so, how useful is this work going to be if the set ops always throw exceptions? If not, then that will mean we have to maintain a

Re: Review Request 62164: GEODE-3566 Moving a bucket during rebalancing does not update overflow stats

2017-09-07 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62164/#review184876 --- Ship it! Ship It! - Darrel Schneider On Sept. 7, 2017, 9:56

Review Request 62178: GEODE-3449: Fix flakiness in ConnectCommandWithSSLTest

2017-09-07 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62178/ --- Review request for geode, Jinmei Liao, Jared Stewart, Ken Howe, Kirk Lund, and

Re: Review Request 62179: Test Rule Fix: clean up client DS when using LocatorServerStartupRule

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62179/ --- (Updated Sept. 7, 2017, 11:12 p.m.) Review request for geode, Jared Stewart,

Re: Geode sessions at SpringOne Platform conference

2017-09-07 Thread Joey McAllister
Hi Jag, Thanks for the pointer to the Geode page for SpringOne. I'd be happy to add that info to the Events section of the Geode Community page. Do you have an existing banner image—preferably one comparable in dimensions to the ApacheCon banner currently on that page, which the SpringOne banner

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

2017-09-07 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #671 was successful. --- Scheduled 2029 tests in total. https://build.spring.io/browse/SGF-NAG-671/ -- This

Re: Review Request 62163: GEODE-3096: pulling in refactoring work on HttpOperationInvoker

2017-09-07 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62163/#review184901 ---

Review Request 62180: refactor away GemfireCacheImpl.getInstance from lucene function

2017-09-07 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62180/ --- Review request for geode, Barry Oglesby and Dan Smith. Bugs: GEODE-3557

Re: Review Request 62180: refactor away GemfireCacheImpl.getInstance from lucene function

2017-09-07 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62180/#review184922 --- This review is confusing. It requires the fix for GEODE-3557

Review Request 62179: Test Rule Fix: clean up client DS when using LocatorServerStartupRule

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62179/ --- Review request for geode, Jared Stewart, Ken Howe, Kirk Lund, and Patrick

Re: Review Request 62132: GEODE-3277: Fix error path constructors of Launcher inner State classess

2017-09-07 Thread Ken Howe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62132/ --- (Updated Sept. 7, 2017, 10:29 p.m.) Review request for geode, Emily Yeh,

Re: Review Request 62179: Test Rule Fix: clean up client DS when using LocatorServerStartupRule

2017-09-07 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62179/#review184915 ---

[DISCUSS] Addition of isValid API to Index interface

2017-09-07 Thread Nabarun Nag
*Proposal:* * Index interface will include an API - isValid() which will return true if the index is still valid / uncorrupted, else will return false if it corrupted / invalid. * gfsh command "list index" will have one more column "Is Valid" which will state the status as "true" or "false". *

Re: Review Request 62178: GEODE-3449: Fix flakiness in ConnectCommandWithSSLTest

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62178/#review184939 --- Ship it! Ship It! - Jinmei Liao On Sept. 7, 2017, 10:21

Re: Review Request 62178: GEODE-3449: Fix flakiness in ConnectCommandWithSSLTest

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62178/#review184940 --- Ship it! Ship It! - Jinmei Liao On Sept. 7, 2017, 10:21

Re: Review Request 62132: GEODE-3277: Fix error path constructors of Launcher inner State classess

2017-09-07 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62132/#review184942 ---

Re: Review Request 62179: Test Rule Fix: clean up client DS when using LocatorServerStartupRule

2017-09-07 Thread Jinmei Liao
> On Sept. 7, 2017, 11:34 p.m., Jared Stewart wrote: > > geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java > > Lines 107 (patched) > > > > > > How do you think this reads this