Re: Review Request 48661: GEODE-1543: Changed the log level from error to warn

> On June 13, 2016, 9:33 p.m., Dan Smith wrote: > > If this is expected behavior, I think this message should be info level or > > lower. warning still indicates that something is going wrong. > > Jason Huynh wrote: > I agree, probably should be info for this. > > nabarun nag wrote: >

Re: Review Request 48432: GEODE-1494: Allowing stats to be measured with callbacks

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48432/#review137430 --- Ship it! Ship It! - xiaojian zhou On June 9, 2016, 6:41

Re: OplogCompactor question

It will only run if it can find at least one oplog that is ready for compaction. Once "getOplogsToBeCompacted()" returns null then a new compaction task will not be scheduled until a new oplog becomes compactable (see

[GitHub] incubator-geode issue #154: GEODE-1470: Upgrade log4j to 2.6

Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/154 Updated to 2.6.1 and fixed the failing unit tests. --- 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

Review Request 48667: GEODE-1544: Fail to initiate final check when network partition occurs

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

Re: Review Request 48383: fix IllegalStateException from importNewObject

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

Re: Review Request 48652: GEODE-1517: Transaction could still proceed after TXManagerImpl is closing during cache close

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48652/#review137422 --- Ship it! Ship It! - Darrel Schneider On June 13, 2016, 9:58

Re: Review Request 48383: fix IllegalStateException from importNewObject

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

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48665/ --- Review request for geode, Hitesh Khamesra, Jianxia Chen, and Udo Kohlmeyer.

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

--- Spring Data GemFire > Nightly-ApacheGeode > #339 was successful. --- Scheduled 1409 tests in total. https://build.spring.io/browse/SGF-NAG-339/ -- This

Re: Review Request 48661: GEODE-1543: Changed the log level from error to warn

> On June 13, 2016, 9:33 p.m., Dan Smith wrote: > > If this is expected behavior, I think this message should be info level or > > lower. warning still indicates that something is going wrong. > > Jason Huynh wrote: > I agree, probably should be info for this. > > nabarun nag wrote: >

Re: Review Request 48661: GEODE-1543: Changed the log level from error to warn

> On June 13, 2016, 9:33 p.m., Dan Smith wrote: > > If this is expected behavior, I think this message should be info level or > > lower. warning still indicates that something is going wrong. > > Jason Huynh wrote: > I agree, probably should be info for this. I was wondering that the

Re: Review Request 48661: GEODE-1543: Changed the log level from error to warn

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48661/#review137390 --- If this is expected behavior, I think this message should be info

Fwd: Low-hanging fruit - Geode

FYI. Thanks, Roman. -- Forwarded message -- From: Paul Merlin Date: Mon, Jun 13, 2016 at 12:54 AM Subject: Re: Low-hanging fruit - Geode To: d...@zest.apache.org Niclas Hedhman a écrit : > Hi, > At ApacheCon I ran into Apache Geode (incubating) people. That

Review Request 48661: GEODE-1543: Changed the log level from error to warn

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48661/ --- Review request for geode, anilkumar gingade, Jason Huynh, Dan Smith, and

Re: Review Request 48383: fix IllegalStateException from importNewObject

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48383/#review137330 --- Ship it! The description of the fix is somewhat misleading

Re: Review Request 48658: Change from MultiFieldQueryParser to StandardQueryParser

> On June 13, 2016, 6:08 p.m., Dan Smith wrote: > > I don't think we want to use the first field as the default field long > > term. That's behavior that might be confusing to users, especially since > > the order of the fields is not really defined if you pass in a map of > >

Re: Review Request 48431: GEODE-1494: Changing stats list to be a CopyOnWriteArrayList

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48431/#review137346 --- Ship it! This change looks good. I think your change makes

Re: Review Request 48658: Change from MultiFieldQueryParser to StandardQueryParser

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48658/#review137345 --- Ship it! I don't think we want to use the first field as the

Review Request 48658: Change from MultiFieldQueryParser to StandardQueryParser

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

Re: Review Request 48432: GEODE-1494: Allowing stats to be measured with callbacks

> On June 10, 2016, 7:34 p.m., anilkumar gingade wrote: > > geode-core/src/main/java/com/gemstone/gemfire/internal/HostStatSampler.java, > > line 55 > > > > > > Is it intentional...We are chaging this from 3 to 30

Re: Review Request 48431: GEODE-1494: Changing stats list to be a CopyOnWriteArrayList

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48431/#review137333 --- Ship it! Ship It! - Kirk Lund On June 8, 2016, 6:06 p.m.,

Review Request 48652: GEODE-1517: Transaction could still proceed after TXManagerImpl is closing during cache close

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48652/ --- Review request for geode, Darrel Schneider and Swapnil Bawaskar. Bugs:

[GitHub] incubator-geode issue #154: GEODE-1470: Upgrade log4j to 2.6

Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/154 2.6.1 was released on June 5th, so we should upgrade to 2.6.1 instead of 2.6. The following tests fail consistently with both 2.6 and 2.6.1: *

Build failed in Jenkins: Geode-spark-connector #27

See -- [...truncated 474 lines...] Download https://repo1.maven.org/maven2/org/hibernate/hibernate-annotations/3.5.5-Final/hibernate-annotations-3.5.5-Final.pom Download

Jenkins build is back to normal : Geode-nightly #498

See