Re: New committer and PPMC Member: Nabarun Nag

2016-06-14 Thread Nitin Lamba
Congrats and welcome! On Tue, Jun 14, 2016 at 3:33 PM, Jason Huynh wrote: > Congrats! > > On Tue, Jun 14, 2016 at 3:22 PM, Dave Barnes wrote: > > > Welcome, Nabarun! > > > > On Tue, Jun 14, 2016 at 3:07 PM, Dan Smith > >

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

2016-06-14 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #340 was successful. --- Scheduled 1409 tests in total. https://build.spring.io/browse/SGF-NAG-340/ -- This

Re: New committer and PPMC Member: Nabarun Nag

2016-06-14 Thread Jason Huynh
Congrats! On Tue, Jun 14, 2016 at 3:22 PM, Dave Barnes wrote: > Welcome, Nabarun! > > On Tue, Jun 14, 2016 at 3:07 PM, Dan Smith > wrote: > > > Please welcome the newest committer to the apache Geode project - Nabarun > > Nag! > > > > We

Re: New committer and PPMC Member: Nabarun Nag

2016-06-14 Thread Dave Barnes
Welcome, Nabarun! On Tue, Jun 14, 2016 at 3:07 PM, Dan Smith wrote: > Please welcome the newest committer to the apache Geode project - Nabarun > Nag! > > We appreciate all the work he has already contributed thus far and are > looking forward to continued

Re: New committer and PPMC Member: Nabarun Nag

2016-06-14 Thread Joey McAllister
Nice! Congrats, Naba! On Tue, Jun 14, 2016 at 3:07 PM Dan Smith wrote: > Please welcome the newest committer to the apache Geode project - Nabarun > Nag! > > We appreciate all the work he has already contributed thus far and are > looking forward to continued

[GitHub] incubator-geode issue #156: GEODE-1543: Change the log level from error to i...

2016-06-14 Thread upthewaterspout
Github user upthewaterspout commented on the issue: https://github.com/apache/incubator-geode/pull/156 +1 --- 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,

Re: Review Request 48701: add a test to query json object from geode-lucene

2016-06-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48701/#review137592 --- Ship it! Ship It! - Dan Smith On June 14, 2016, 9:56 p.m.,

New committer and PPMC Member: Nabarun Nag

2016-06-14 Thread Dan Smith
Please welcome the newest committer to the apache Geode project - Nabarun Nag! We appreciate all the work he has already contributed thus far and are looking forward to continued involvement with Geode as a committer and PPMC member. -Dan

Re: Review Request 48701: add a test to query json object from geode-lucene

2016-06-14 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48701/ --- (Updated June 14, 2016, 9:56 p.m.) Review request for geode and Dan Smith.

[GitHub] incubator-geode pull request #156: GEODE-1543: Change the log level from err...

2016-06-14 Thread nabarunnag
GitHub user nabarunnag opened a pull request: https://github.com/apache/incubator-geode/pull/156 GEODE-1543: Change the log level from error to info * When multiple peers join the cluster conncurrently, only the first one that joins will distribute the index creation and other will

Re: Review Request 48699: GEODE-11: Support indexing values that are Strings or Numbers

2016-06-14 Thread Jason Huynh
> On June 14, 2016, 5:07 p.m., Jason Huynh wrote: > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/serializer/HeterogeneousLuceneSerializer.java, > > line 71 > > > > > > If they

Re: Review Request 48699: GEODE-11: Support indexing values that are Strings or Numbers

2016-06-14 Thread Dan Smith
> On June 14, 2016, 5:07 p.m., Jason Huynh wrote: > > geode-lucene/src/main/java/com/gemstone/gemfire/cache/lucene/internal/repository/serializer/HeterogeneousLuceneSerializer.java, > > line 71 > > > > > > If they

Re: Review Request 48701: add a test to query json object from geode-lucene

2016-06-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48701/#review137564 --- Nice. Maybe it would be better to have a different method to

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

2016-06-14 Thread Dick Cavender
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47157/#review137567 --- Ship it! Ship It! - Dick Cavender On June 14, 2016, 4:58

Review Request 48701: add a test to query json object from geode-lucene

2016-06-14 Thread xiaojian zhou
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48701/ --- Review request for geode and Dan Smith. Bugs: GEODE-1545

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

2016-06-14 Thread Dan Smith
> On May 10, 2016, 4:09 p.m., Dan Smith wrote: > > I thought we were going to continue to make precheckin run these tests. You > > should make precheckin depend on flakyTest. > > > > Weren't we going to run these with forkEvery 1? The ingregration tests run > > with forkEvery 1, and the

Re: Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48695/#review137557 --- Ship it! Ship It! - Dan Smith On June 14, 2016, 5:41 p.m.,

Re: Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48695/ --- (Updated June 14, 2016, 5:41 p.m.) Review request for geode, anilkumar

Re: Review Request 48556: Fixing spark connector build issue due to ConfigurationProperties name change

2016-06-14 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48556/#review137554 --- Ship it! Ship It! - Udo Kohlmeyer On June 10, 2016, 3:12

Re: Review Request 48665: GEODE-1542 shared/unordered tcp/ip connection times out, initiating suspicion

2016-06-14 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48665/#review137552 --- Ship it! Ship It! - Udo Kohlmeyer On June 13, 2016, 10:50

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

2016-06-14 Thread Mark Bretl
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47157/#review137547 --- Ship it! Ship It! - Mark Bretl On June 14, 2016, 9:58 a.m.,

Re: Review Request 48699: GEODE-11: Support indexing values that are Strings or Numbers

2016-06-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48699/#review137544 ---

Re: Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Jason Huynh
> On June 14, 2016, 4:55 p.m., Dan Smith wrote: > > Hi Jason, > > > > It looks good, but I think the LuceneQueryProvider should not be passed a > > default field. The APIs on LuceneQueryFactory should be: > > > > create(String index, String region, String query, String defaultField) > >

Review Request 48699: GEODE-11: Support indexing values that are Strings or Numbers

2016-06-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48699/ --- Review request for geode, anilkumar gingade, Jason Huynh, nabarun nag, and

Re: Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Jason Huynh
> On June 14, 2016, 4:55 p.m., Dan Smith wrote: > > Hi Jason, > > > > It looks good, but I think the LuceneQueryProvider should not be passed a > > default field. The APIs on LuceneQueryFactory should be: > > > > create(String index, String region, String query, String defaultField) > >

Re: Review Request 47157: GEODE-1374: Add a flakyTest build task

2016-06-14 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47157/ --- (Updated June 14, 2016, 4:58 p.m.) Review request for geode, Dick Cavender,

Re: Review Request 48665: GEODE-1542 shared/unordered tcp/ip connection times out, initiating suspicion

2016-06-14 Thread Bruce Schuchardt
> On June 14, 2016, 4:47 p.m., Jianxia Chen wrote: > > geode-core/src/main/java/com/gemstone/gemfire/distributed/internal/membership/gms/fd/GMSHealthMonitor.java, > > line 566 > > > > > > Do you want to keep the

Re: Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48695/#review137537 --- Hi Jason, It looks good, but I think the LuceneQueryProvider

Re: Review Request 48665: GEODE-1542 shared/unordered tcp/ip connection times out, initiating suspicion

2016-06-14 Thread Jianxia Chen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48665/#review137535 --- Fix it, then Ship it!

Re: Review Request 48665: GEODE-1542 shared/unordered tcp/ip connection times out, initiating suspicion

2016-06-14 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48665/#review137534 --- Ship it! Ship It! - Hitesh Khamesra On June 13, 2016, 10:50

Review Request 48695: Allows passing in a defaultField into the query search

2016-06-14 Thread Jason Huynh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48695/ --- Review request for geode, anilkumar gingade, nabarun nag, Dan Smith, and

Jenkins build is back to normal : Geode-spark-connector #28

2016-06-14 Thread Apache Jenkins Server
See