Re: Review Request 45993: GEODE-1201: Adding compileRuntimeLibs to geode-assembly for tests Amending bind address configuration for http-service-bind-address

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45993/#review128808 ---

Re: [VOTE] RC2: Apache Geode (Incubating) second Milestone release - v1.0.0-incubating.M2

- Checked out tag, built from source - Built from source distribution - Ran from binary distribution - Checked hashes and signature - Reviewed maven repo - Checked against a few sample apps [1] [2] - Deployed to AWS [3] +1 Anthony [1] https://github.com/metatype/geode-hello-world [2]

Re: Review Request 46156: GEODE-679 Explore removing SocketIOWithTimeout and other classes related to FD soft leak

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46156/#review128805 --- SocketUtils is not in the diff. I assume you will remove both

Re: [VOTE] RC2: Apache Geode (Incubating) second Milestone release - v1.0.0-incubating.M2

Thanks Kareem! You're right, the first line should read: This is second release candidate for Apache Geode, version 1.0.0-incubating.M2. -Dan On Wed, Apr 13, 2016 at 4:01 PM, kareem shabazz wrote: > Shouldn't that first line be M2 rather than M1? > > -- > Kareem > >

Re: [VOTE] RC2: Apache Geode (Incubating) second Milestone release - v1.0.0-incubating.M2

Shouldn't that first line be M2 rather than M1? -- Kareem _ From: Dan Smith Sent: Wednesday, April 13, 2016 6:55 PM Subject: [VOTE] RC2: Apache Geode (Incubating) second Milestone release - v1.0.0-incubating.M2 To: geode

[VOTE] RC2: Apache Geode (Incubating) second Milestone release - v1.0.0-incubating.M2

This is second release candidate for Apache Geode, version 1.0.0-incubating.M1. *** Please download, test and vote by Monday, Apr 18, 0800 PST If you are wondering why you didn't see RC1, it's because I caught a mistake after I created the tag but before sending out the vote. Thanks for all the

[GitHub] incubator-geode pull request: GEODE-1032 : Placed additional wait ...

GitHub user nabarunnag opened a pull request: https://github.com/apache/incubator-geode/pull/128 GEODE-1032 : Placed additional wait time to check for empty queue, refactored WANTestBase.java to remove unused functions, replaced wait criterions with awaitility. GEODE-1032 : Placed

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

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

Re: Review Request 46156: GEODE-679 Explore removing SocketIOWithTimeout and other classes related to FD soft leak

> On April 13, 2016, 6:01 p.m., Jason Huynh wrote: > > I think we can also delete SocketIOWithTimeout, SocketInputStream and > > SocketOutputStream along with these changes I'll remove those and SocketInputWrapper as well. There is only one reference to these classes, in

Re: Review Request 45993: GEODE-1201: Adding compileRuntimeLibs to geode-assembly for tests Amending bind address configuration for http-service-bind-address

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45993/ --- (Updated April 13, 2016, 7:39 p.m.) Review request for geode, Bruce

Re: Review Request 46156: GEODE-679 Explore removing SocketIOWithTimeout and other classes related to FD soft leak

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46156/#review128717 --- I think we can also delete SocketIOWithTimeout, SocketInputStream

Review Request 46156: GEODE-679 Explore removing SocketIOWithTimeout and other classes related to FD soft leak

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

Re: Jenkins for release branches?

I think the thing to do is not call uploadArtifacts on the release branch jenkins job. It looks like we have some build logic that targets a different maven repo if you are on a release branch. We don't want to upload to any maven repo for this job. -Dan On Wed, Apr 13, 2016 at 10:08 AM, Anthony

Re: Review Request 46154: GEODE-1018: Retrieve queues only once gateway listener to avoid a race

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46154/#review128712 --- Ship it! Ship It! - anilkumar gingade On April 13, 2016,

Review Request 46154: GEODE-1018: Retrieve queues only once gateway listener to avoid a race

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46154/ --- Review request for geode, Barry Oglesby and nabarun nag. Repository: geode

Re: Review Request 46060: fix the off-heap leak in netWrite

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46060/#review128705 --- Ship it! Ship It! - Ken Howe On April 11, 2016, 11:05 p.m.,

Re: Jenkins for release branches?

+1 - I'm starting to work on that but the release branch currently has signing requirements and we may need to remove that, otherwise Jenkins will fail A GnuPG key ID is required for signing. Please set signing.keyId= in /.gradle/gradle.properties On Wed, Apr 13, 2016 at 9:33 AM

Re: Review Request 46123: GEODE-1198 CI Failure: DistributedSystemDUnitTest.testConflictingUDPPort

> On April 13, 2016, 12:16 a.m., Hitesh Khamesra wrote: > > geode-core/src/test/java/com/gemstone/gemfire/distributed/DistributedSystemDUnitTest.java, > > line 394 > > > > > > When test pases, from where it gets

Re: Review Request 46123: GEODE-1198 CI Failure: DistributedSystemDUnitTest.testConflictingUDPPort

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46123/ --- (Updated April 13, 2016, 4:58 p.m.) Review request for geode, Bruce

Build failed in Jenkins: Geode--release #1

See -- Started by user markito [EnvInject] - Loading node environment variables. Building remotely on ubuntu-1 (docker Ubuntu ubuntu yahoo-not-h2 ubuntu1) in workspace

Re: Jenkins for release branches?

+1 On Tue, Apr 12, 2016 at 2:53 PM, Dan Smith wrote: > Hi all, > > Can we create a jenkins job that runs on the release branch? We can > probably just configure a job to look for any branches that start with > release-* and do a build on them if there are changes. > > -Dan >

Review Request 46151: possible fix for intermittent failure of testTombstones

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46151/ --- Review request for geode, Eric Shu, Scott Jewell, Ken Howe, and Sai Boorlagadda.

Re: Jenkins for release branches?

+1 On Tue, Apr 12, 2016 at 10:01 PM, Rajiv Kumar wrote: > +1 > > On Wed, Apr 13, 2016 at 3:23 AM, Dan Smith wrote: > > > Hi all, > > > > Can we create a jenkins job that runs on the release branch? We can > > probably just configure a job to look for any

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

See

Comparing maven deps vs lib/ dir

+1 Great progress indeed!! On Tuesday, April 12, 2016, Anthony Baker wrote: > See [1]. Only trouble I had was excluding log4j-slf4j-impl. Without that > I hit a startup error complaining about logging conflicts. > > compile