Re: Experimental features for 1.0

2016-08-10 Thread Dave Barnes
JVSD is not fully-featured, yet. Does it belong on the list? > On Aug 10, 2016, at 6:26 PM, Kirk Lund wrote: > > +1 for Jira tickets with "experimental" label > > Also, don't forget we have the @Experimental annotation defined in > geode-common that should be applied to any

Re: Experimental features for 1.0

2016-08-10 Thread Kirk Lund
+1 for Jira tickets with "experimental" label Also, don't forget we have the @Experimental annotation defined in geode-common that should be applied to any experimental user APIs before final 1.0.0 release. -Kirk On Wednesday, August 10, 2016, William Markito wrote: > As

Re: Experimental features for 1.0

2016-08-10 Thread John Blum
+1 What about HDFS storage support, or was this dropped completely? On Wed, Aug 10, 2016 at 6:12 PM, Joey McAllister wrote: > +1 - Great idea > > On Wed, Aug 10, 2016 at 6:10 PM Michael Stolz wrote: > > > Yes we should definitely do this > > > > -- >

Re: Experimental features for 1.0

2016-08-10 Thread Joey McAllister
+1 - Great idea On Wed, Aug 10, 2016 at 6:10 PM Michael Stolz wrote: > Yes we should definitely do this > > -- > Mike Stolz > Principal Engineer - Gemfire Product Manager > Mobile: 631-835-4771 > On Aug 10, 2016 9:01 PM, "William Markito" wrote: > > > As

Re: Experimental features for 1.0

2016-08-10 Thread Michael Stolz
Yes we should definitely do this -- Mike Stolz Principal Engineer - Gemfire Product Manager Mobile: 631-835-4771 On Aug 10, 2016 9:01 PM, "William Markito" wrote: > As we move forward to 1.0 I'd like to propose creating JIRAS with the > "experimental" label to capture

Experimental features for 1.0

2016-08-10 Thread William Markito
As we move forward to 1.0 I'd like to propose creating JIRAS with the "experimental" label to capture everything we have that is classified as such. This would help users and projects consume Geode to consider those features properly and decide to add support (or not) to them as well. AFAIK

Re: Revised SSL properties failure scenario advice.

2016-08-10 Thread Kirk Lund
+1 to introduce "ssl-default-alias" and fail if it's not set for multi-key keystore On Wed, Aug 10, 2016 at 4:34 PM, Bruce Schuchardt wrote: > +1 for ssl-default-alias and failing > > > Le 8/10/2016 à 3:36 PM, Udo Kohlmeyer a écrit : > >> Hi there guys, >> >> As per the

Re: Revised SSL properties failure scenario advice.

2016-08-10 Thread Bruce Schuchardt
+1 for ssl-default-alias and failing Le 8/10/2016 à 3:36 PM, Udo Kohlmeyer a écrit : Hi there guys, As per the proposal for the revision of the SSL configuration , I'm nearing the completion of this feature. What I

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

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

Revised SSL properties failure scenario advice.

2016-08-10 Thread Udo Kohlmeyer
Hi there guys, As per the proposal for the revision of the SSL configuration , I'm nearing the completion of this feature. What I have come across is some scenarios where a system is configured with a multi-key

[GitHub] incubator-geode pull request #230: GEODE-1744: Probable bugs from == use fix...

2016-08-10 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/230#discussion_r74340943 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/management/internal/cli/AbstractCliAroundInterceptor.java --- @@ -88,7 +88,7 @@ protected

Re: Review Request 49962: Removed extra fields from distributedmember while serialization/de

2016-08-10 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49962/#review145409 --- It looks good but you need to change the GMSMember reference in

[GitHub] incubator-geode pull request #230: GEODE-1744: Probable bugs from == use fix...

2016-08-10 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/230#discussion_r74340020 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/admin/AlertLevel.java --- @@ -145,8 +147,8 @@ public boolean equals(Object other) {

[GitHub] incubator-geode issue #231: Feature/geode 1743

2016-08-10 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/231 LGTM +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, or

[GitHub] incubator-geode pull request #230: GEODE-1744: Probable bugs from == use fix...

2016-08-10 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/230#discussion_r74338007 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/admin/AlertLevel.java --- @@ -145,8 +147,8 @@ public boolean equals(Object other) {

[GitHub] incubator-geode pull request #230: GEODE-1744: Probable bugs from == use fix...

2016-08-10 Thread metatype
Github user metatype commented on a diff in the pull request: https://github.com/apache/incubator-geode/pull/230#discussion_r74339073 --- Diff: geode-core/src/main/java/com/gemstone/gemfire/admin/AlertLevel.java --- @@ -145,8 +147,8 @@ public boolean equals(Object other) {

[GitHub] incubator-geode issue #230: GEODE-1744: Probable bugs from == use fixed

2016-08-10 Thread metatype
Github user metatype commented on the issue: https://github.com/apache/incubator-geode/pull/230 Is there a test that exposes the probable bug and verifies that the issue is fixed? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

Re: Review Request 49962: Removed extra fields from distributedmember while serialization/de

2016-08-10 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49962/ --- (Updated Aug. 10, 2016, 9:25 p.m.) Review request for geode, Bruce Schuchardt

[GitHub] incubator-geode pull request #231: Feature/geode 1743

2016-08-10 Thread joeymcallister
GitHub user joeymcallister opened a pull request: https://github.com/apache/incubator-geode/pull/231 Feature/geode 1743 Add Mike Stolz to committers You can merge this pull request into a Git repository by running: $ git pull https://github.com/joeymcallister/incubator-geode

[GitHub] incubator-geode pull request #230: GEODE-1744: Probable bugs from == use fix...

2016-08-10 Thread gracemeilen
GitHub user gracemeilen opened a pull request: https://github.com/apache/incubator-geode/pull/230 GEODE-1744: Probable bugs from == use fixed You can merge this pull request into a Git repository by running: $ git pull https://github.com/gracemeilen/incubator-geode

Re: Review Request 50962: GEODE-1742: Fixing synchronization of cqListeners in CqAttributesImpl

2016-08-10 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50962/#review145401 --- Ship it! Ship It! - anilkumar gingade On Aug. 10, 2016,

Re: Review Request 50903: GEODE-1583: Always pass the keep alive flag when destroying a connection

2016-08-10 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50903/#review145399 --- Ship it! Ship It! - anilkumar gingade On Aug. 10, 2016,

Re: Review Request 50903: GEODE-1583: Always pass the keep alive flag when destroying a connection

2016-08-10 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50903/ --- (Updated Aug. 10, 2016, 8:39 p.m.) Review request for geode and anilkumar

Re: Geode M3 docs: Review opportunity

2016-08-10 Thread William Markito
LGTM. +1 Thanks Dave! On Mon, Aug 8, 2016 at 11:40 AM, Dave Barnes wrote: > I've posted the Geode User docs for the upcoming M3 release here: > > http://geode-review.cfapps.io/ > > Please have a look at the section(s) describing the features you've > implemented and send

Review Request 50962: GEODE-1742: Fixing synchronization of cqListeners in CqAttributesImpl

2016-08-10 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50962/ --- Review request for geode and anilkumar gingade. Repository: geode

Build failed in Jenkins: Geode-nightly #556

2016-08-10 Thread Apache Jenkins Server
See Changes: [gzhou] This closes #229 -- [...truncated 509 lines...] :geode-wan:javadocJar :geode-wan:sourcesJar :geode-wan:signArchives SKIPPED :geode-web:javadoc UP-TO-DATE :geode-web:javadocJar