Re: Review Request 51394: GEODE-1372 Geode UDP communications are not secure when SSL is configured

2016-08-24 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51394/#review146723 --- I have a few review comments, and I think we need you to walk us

Re: Review Request 51380: GEODE-1809: Refactor GemFireVersion

2016-08-24 Thread Dick Cavender
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51380/#review146728 --- Ship it! Built this on open side fine and manually inspected

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

2016-08-24 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #411 was successful. --- Scheduled 1441 tests in total. https://build.spring.io/browse/SGF-NAG-411/ -- This

Review Request 51394: GEODE-1372 Geode UDP communications are not secure when SSL is configured

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

[GitHub] incubator-geode issue #201: GEODE-1617: Regions can be created with a variet...

2016-08-24 Thread joeymcallister
Github user joeymcallister commented on the issue: https://github.com/apache/incubator-geode/pull/201 @seankarson @kjduling The document has been updated. Thanks for the heads up! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

Re: Review Request 51385: GEODE-900 test endup uding old cache

2016-08-24 Thread Bruce Schuchardt
> On Aug. 24, 2016, 8:27 p.m., Bruce Schuchardt wrote: > > While it's okay to synchronize like that in checkExistingCache() I think it > > might be more appropriate to synchronize on CacheFactory.class in > > basicCreate() and fold the checkExistingClass() method into that method > > since

Re: Review Request 51385: GEODE-900 test endup uding old cache

2016-08-24 Thread Hitesh Khamesra
> On Aug. 24, 2016, 8:27 p.m., Bruce Schuchardt wrote: > > While it's okay to synchronize like that in checkExistingCache() I think it > > might be more appropriate to synchronize on CacheFactory.class in > > basicCreate() and fold the checkExistingClass() method into that method > > since

Re: Review Request 51385: GEODE-900 test endup uding old cache

2016-08-24 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51385/#review146705 --- While it's okay to synchronize like that in checkExistingCache()

Re: Review Request 51385: GEODE-900 test endup uding old cache

2016-08-24 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51385/#review146696 --- Ship it! Ship It! - Udo Kohlmeyer On Aug. 24, 2016, 6:30

Review Request 51385: GEODE-900 test endup uding old cache

2016-08-24 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51385/ --- Review request for geode, Bruce Schuchardt and Darrel Schneider. Repository:

Review Request 51380: GEODE-1809: Refactor GemFireVersion

2016-08-24 Thread Anthony Baker
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51380/ --- Review request for geode, Dick Cavender and Jens Deppe. Repository: geode

[GitHub] incubator-geode issue #201: GEODE-1617: Regions can be created with a variet...

2016-08-24 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/201 Good catch. @davebarnes97 @karensmolermiller Can either of you follow up on this? Looks like we've a page out of date. --- If your project is set up for it, you can reply to

[GitHub] incubator-geode issue #201: GEODE-1617: Regions can be created with a variet...

2016-08-24 Thread seankarson
Github user seankarson commented on the issue: https://github.com/apache/incubator-geode/pull/201 The example in the docs for how to create a dynamic region on the server no longer works in M3 due to this fix.

Re: Brew formulae for geode

2016-08-24 Thread Kevin Duling
I'll be updating MacPorts probably this week, too, also calling it apache-geode. On Wed, Aug 24, 2016 at 7:43 AM, Anthony Baker wrote: > Bumping this old thread: > > I’ve updated the brew formula with the 1.0.0-incubating.M3 release and > renamed it to apache-geode. > > `brew

Build failed in Jenkins: Geode-nightly #570

2016-08-24 Thread Apache Jenkins Server
See Changes: [upthewaterspout] GEODE-1810: WAN locator exchange doesn't include hostname-for-clients if [upthewaterspout] GEODE-1811: Marking runtime dependencies as optional in the pom [upthewaterspout] GEODE-1814: Create the gateway

Re: Brew formulae for geode

2016-08-24 Thread Anthony Baker
Bumping this old thread: I’ve updated the brew formula with the 1.0.0-incubating.M3 release and renamed it to apache-geode. `brew install apache-geode` Anthony > On Apr 27, 2016, at 6:51 AM, Anthony Baker wrote: > > There’s a clarification thread here [1]. I’ll followup