Need to enable authentication in Locator JMX

2017-07-14 Thread Dinesh Akhand
Hi Team, I need to enable the JMX authentication in geode for clients like pulse . could you please suggest what is correct procedure. I did the below things: JMX is enabled: jmx-manager-start=true jmx-manager=true Add entry Locator.properties:

Review Request 60873: GEODE-2189 Docs: Update Swagger UI links Added link to OpenAPI specification.

2017-07-14 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60873/ --- Review request for geode. Repository: geode Description --- GEODE-2189

[GitHub] geode pull request #635: GEODE-2594 Remove tools.jar and --pid options from ...

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/635 --- 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 if the feature is

[GitHub] geode pull request #633: GEODE-3170: Closed socket doesn't result in an infi...

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/633 --- 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 if the feature is

[GitHub] geode pull request #638: Feature/geode 2860

2017-07-14 Thread nreich
GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/638 Feature/geode 2860 Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken:

[GitHub] geode pull request #633: GEODE-3170: Closed socket doesn't result in an infi...

2017-07-14 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/633#discussion_r127540044 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/GenericProtocolServerConnection.java --- @@ -68,9 +56,8 @@ protected

Re: Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-14 Thread Dan Smith
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60875/#review180588 --- I like where you are going with this and I think this will make

[GitHub] geode pull request #631: GEODE-3051: Remove unreachable exception handling i...

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/631 --- 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 if the feature is

Re: Stored procedures on Apache Geode.

2017-07-14 Thread Michael Stolz
Pivotal provides a closed-source connector to link commercial GemFire to commercial Greenplum. Pivotal has no intention of opening that connector at this time. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Fri, Jul 14, 2017 at 2:33 AM, marios390

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

2017-07-14 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #616 was successful. --- Scheduled 1955 tests in total. https://build.spring.io/browse/SGF-NAG-616/ -- This

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Kenneth Howe
+1 Downloaded and built from source. Basic operations from gfsh - create cluster simple puts and queries > On Jul 14, 2017, at 12:45 PM, Bruce Schuchardt wrote: > > +1 > > downloaded apache-geode-1.2.0.tar.gz > > ran through the 15-minute startup guide with no

[GitHub] geode pull request #637: GEODE-2612 Document new import data option

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/637 --- 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 if the feature is

[GitHub] geode issue #633: GEODE-3170: Closed socket doesn't result in an infinite lo...

2017-07-14 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/633 @hiteshk25 @pivotal-amurmann --- 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

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Fred Krone
+1 Downloaded and built from source. gfsh -- new string partition resolver for region. Worked. 10/10 would partition resolve again. On Fri, Jul 14, 2017 at 1:08 PM, Kenneth Howe wrote: > +1 > > Downloaded and built from source. Basic operations from gfsh - create > cluster

[GitHub] geode pull request #633: GEODE-3170: Closed socket doesn't result in an infi...

2017-07-14 Thread galen-pivotal
Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/633#discussion_r127550540 --- Diff: geode-core/src/main/java/org/apache/geode/internal/cache/tier/sockets/GenericProtocolServerConnection.java --- @@ -68,9 +56,8 @@ protected

Re: Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60874/#review180569 ---

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Bruce Schuchardt
+1 downloaded apache-geode-1.2.0.tar.gz ran through the 15-minute startup guide with no issues On 7/12/17 7:58 AM, Anthony Baker wrote: This is the second release candidate for Apache Geode, version 1.2.0. Thanks to all the community members for their contributions to this release! This

[GitHub] geode pull request #632: GEODE-3203: fixing protobuf build.gradle to respect...

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/632 --- 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 if the feature is

Re: No nulls

2017-07-14 Thread John Blum
+1 as well. However, I will caution this... use Java 8's new java.util.Optional class in your codebase judiciously. Using it everywhere, especially on critical code paths can and most likely will affect your performance. Internally, j.u.Optional allocates new objects for nearly every operation

[GitHub] geode issue #630: GEODE-3141: GetRegion Operation implemented

2017-07-14 Thread galen-pivotal
Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/630 Looks like Spotless is failing in CI. --- 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

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Jared Stewart
+1 Built from src and ran some basic operations on a small cluster. > On Jul 14, 2017, at 11:30 AM, Dick Cavender wrote: > > +1 > > Reviewed 1.2.0 javadocs and Geode docs > > Ubuntu 12.04 & JDK8 Update 111 > >Downloaded src dist and successfully built >

[GitHub] geode pull request #636: GEODE-3198: Allow bucket creation when over local-m...

2017-07-14 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/636 --- 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 if the feature is

[GitHub] geode issue #634: Feature/geode 3175

2017-07-14 Thread bschuchardt
Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/634 @kohlmu-pivotal @hiteshk25 @galen-pivotal @pivotal-amurmann --- 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

Re: Review Request 60834: GEODE-3051: Remove unreachable exception handling in AcceptorImpl.accept

2017-07-14 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60834/#review180578 --- Ship it! Ship It! - Udo Kohlmeyer On July 13, 2017, 12:35

Re: Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Eric Shu
> On July 14, 2017, 7:07 p.m., anilkumar gingade wrote: > > geode-core/src/main/java/org/apache/geode/internal/cache/AbstractRegionMap.java > > Lines 2515 (patched) > > > > > > Is't this means the invalidate failed

Review Request 60881: GEODE-3207 Swagger library updates: update user guide

2017-07-14 Thread Dave Barnes
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60881/ --- Review request for geode and Jared Stewart. Repository: geode Description

Re: Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread anilkumar gingade
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60874/#review180570 ---

[GitHub] geode pull request #635: GEODE-2594 Remove tools.jar and --pid options from ...

2017-07-14 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/635#discussion_r127515789 --- Diff: geode-docs/configuring/running/running_the_locator.html.md.erb --- @@ -35,7 +35,7 @@ You can run the locator standalone or embedded within another

[GitHub] geode pull request #635: GEODE-2594 Remove tools.jar and --pid options from ...

2017-07-14 Thread kirklund
Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/635#discussion_r127515650 --- Diff: geode-docs/configuring/running/running_the_cacheserver.html.md.erb --- @@ -31,7 +31,7 @@ The Geode server is used primarily for hosting long-lived

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Dick Cavender
+1 Reviewed 1.2.0 javadocs and Geode docs Ubuntu 12.04 & JDK8 Update 111 Downloaded src dist and successfully built Verified full version through gfsh using both binary and built source product Ran though tutorial on binary dist SPARC Solaris 11.2 & JDK8 Update 111

[GitHub] geode pull request #635: GEODE-2594 Remove tools.jar and --pid options from ...

2017-07-14 Thread karensmolermiller
Github user karensmolermiller commented on a diff in the pull request: https://github.com/apache/geode/pull/635#discussion_r127524936 --- Diff: geode-docs/configuring/running/running_the_cacheserver.html.md.erb --- @@ -31,7 +31,7 @@ The Geode server is used primarily for hosting

[GitHub] geode pull request #635: GEODE-2594 Remove tools.jar and --pid options from ...

2017-07-14 Thread karensmolermiller
Github user karensmolermiller commented on a diff in the pull request: https://github.com/apache/geode/pull/635#discussion_r127524976 --- Diff: geode-docs/configuring/running/running_the_locator.html.md.erb --- @@ -35,7 +35,7 @@ You can run the locator standalone or embedded within

[GitHub] geode pull request #636: GEODE-3198: Allow bucket creation when over local-m...

2017-07-14 Thread nreich
GitHub user nreich opened a pull request: https://github.com/apache/geode/pull/636 GEODE-3198: Allow bucket creation when over local-max-memory * the bucket will not be created on a member that is over local max * memory, unless all members have failed to create the bucket

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Jinmei Liao
+1 1) download the executable. 2) run gfsh and start locator, server, and random commands. 3) verify pulse is started and running. On Fri, Jul 14, 2017 at 10:37 AM, Hitesh Khamesra < hitesh...@yahoo.com.invalid> wrote: > +1 > same as Dan!! > > On Wednesday, July 12, 2017, 1:38:44 PM PDT, Dan

Re: No nulls

2017-07-14 Thread John Blum
IMO, I think they are very useful, especially since the IDE (e.g. IntelliJ IDEA) can use them to detect problems (e.g. NPEs) earlier rather than later. For example, *Spring* 5 just introduced 2 new annotations: @Nullable [1] (declaring possible null parameters, returns values or fields) and

[GitHub] geode pull request #637: GEODE-2612 Document new import data option

2017-07-14 Thread karensmolermiller
GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode/pull/637 GEODE-2612 Document new import data option @boglesby @joeymcallister @davebarnes97 Please review. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] geode pull request #637: GEODE-2612 Document new import data option

2017-07-14 Thread joeymcallister
Github user joeymcallister commented on a diff in the pull request: https://github.com/apache/geode/pull/637#discussion_r127525516 --- Diff: geode-docs/tools_modules/gfsh/command-pages/import.html.md.erb --- @@ -40,16 +40,17 @@ Import user data from a file to a region.

RE: Stored procedures on Apache Geode.

2017-07-14 Thread marios390
Hi John, Thank you very much for all this information. I will check it out. Appreciate it!. Thanks MS From: John Blum [via Apache Geode (Incubating) Developers Forum] [ml+s70738n24351...@n6.nabble.com] Sent: Friday, July 14, 2017 6:35 PM To: Marios

Re: Review Request 60881: GEODE-3207 Swagger library updates: update user guide

2017-07-14 Thread Joey McAllister
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60881/#review180581 --- Ship it! Ship It! - Joey McAllister On July 14, 2017, 8:02

Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Eric Shu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60874/ --- Review request for geode, anilkumar gingade, Darrel Schneider, Lynn Gallinat,

[GitHub] geode pull request #616: GEODE-2860: Refactor use of EventTracker

2017-07-14 Thread nreich
Github user nreich closed the pull request at: https://github.com/apache/geode/pull/616 --- 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 if the feature is

Re: No nulls

2017-07-14 Thread Galen O'Sullivan
What's the general opinion of @NotNull annotations? Would they be useful? On Fri, Jul 14, 2017 at 9:23 AM, John Blum wrote: > +1 as well. > > However, I will caution this... use Java 8's new java.util.Optional class > in your codebase judiciously. Using it everywhere,

Review Request 60875: GEODE-2654: Backups can capture different members from different points in time

2017-07-14 Thread Lynn Gallinat
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60875/ --- Review request for geode, anilkumar gingade, Darrel Schneider, and Dan Smith.

Re: [VOTE] Apache Geode release - v1.2.0 RC2

2017-07-14 Thread Hitesh Khamesra
+1 same as Dan!! On Wednesday, July 12, 2017, 1:38:44 PM PDT, Dan Smith wrote: +0 I ran geode-release-check against this project. It all looks good except that the md5sum and sha256 sums tasks don't pass because the

Re: Stored procedures on Apache Geode.

2017-07-14 Thread John Blum
Hi Marios- I may not be the best person to answer this. There is a Pivotal GemFire / Greenplum Connector available for download here... https://network.pivotal.io/products/pivotal-gemfire/#/releases/5376/file_groups/495 More information on the Connector can be found here...

Re: Review Request 60718: GEODE-2997: New flow getAll/putAll

2017-07-14 Thread Alexander Murmann
> On July 13, 2017, 1:40 a.m., Galen O'Sullivan wrote: > > geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/PutAllRequestOperationHandler.java > > Lines 38 (patched) > > > > > > I had

Re: Review Request 60834: GEODE-3051: Remove unreachable exception handling in AcceptorImpl.accept

2017-07-14 Thread Galen O'Sullivan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60834/#review180546 --- Ship it! Ship It! - Galen O'Sullivan On July 13, 2017,

Re: Review Request 60856: GEODE-3052 Need to reset isCoordinator flag in GMSLocator.

2017-07-14 Thread Hitesh Khamesra
> On July 14, 2017, 12:18 a.m., Bruce Schuchardt wrote: > > geode-core/src/main/java/org/apache/geode/distributed/internal/membership/gms/locator/GMSLocator.java > > Lines 288 (patched) > > > > > > remove debug

Re: Review Request 60856: GEODE-3052 Need to reset isCoordinator flag in GMSLocator.

2017-07-14 Thread Hitesh Khamesra
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/ --- (Updated July 14, 2017, 5:06 p.m.) Review request for geode, Alexander

Re: Need to enable authentication in Locator JMX

2017-07-14 Thread Jinmei Liao
Hi, Dinesh, which version of Geode you are using? If you are using the latest one. By simply specify a security-manager in your locator.properties will turn on the authentication for jmx. Please see the instruction on this page:

Re: Review Request 60856: GEODE-3052 Need to reset isCoordinator flag in GMSLocator.

2017-07-14 Thread Udo Kohlmeyer
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/#review180545 --- Ship it! Ship It! - Udo Kohlmeyer On July 13, 2017, 11:28

Re: Review Request 60856: GEODE-3052 Need to reset isCoordinator flag in GMSLocator.

2017-07-14 Thread Bruce Schuchardt
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/#review180549 --- Ship it! Ship It! - Bruce Schuchardt On July 14, 2017,

Re: Review Request 60873: GEODE-2189 Docs: Update Swagger UI links Added link to OpenAPI specification.

2017-07-14 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60873/#review180607 --- Ship it! Ship It! - Karen Miller On July 14, 2017, 5:08

New Committer: Deepak Dixit

2017-07-14 Thread Mark Bretl
The Apache Geode Project Management Committee has invited Deepak Dixit to be committer on the project. We are pleased to announce he has accepted. Please join me in welcoming Deepak! Thanks, Mark On behalf of the Apache Geode PMC

Re: Review Request 60874: GEODE-3204: txApplyInvalidate should not modify region entry with a removed token

2017-07-14 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60874/#review180612 --- Ship it! Ship It! - Darrel Schneider On July 14, 2017, 1:02

Re: New Committer: Deepak Dixit

2017-07-14 Thread Anthony Baker
Welcome Deepak! Anthony Sent from my iPhone > On Jul 14, 2017, at 4:46 PM, Mark Bretl wrote: > > The Apache Geode Project Management Committee has invited Deepak Dixit to > be committer on the project. We are pleased to announce he has accepted. > > Please join me in

Passed: apache/geode#3162 (develop - 9905794)

2017-07-14 Thread Travis CI
Build Update for apache/geode - Build: #3162 Status: Passed Duration: 22 minutes and 4 seconds Commit: 9905794 (develop) Author: Galen O'Sullivan Message: GEODE-3170: Closed socket doesn't result in an infinite loop. This closes #633 * Protobuf

Re: Review Request 60881: GEODE-3207 Swagger library updates: update user guide

2017-07-14 Thread Karen Miller
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60881/#review180608 --- Ship it! Ship It! - Karen Miller On July 14, 2017, 8:02

RE: Query for JMX "Async Event Queue properties"

2017-07-14 Thread Dinesh Akhand
Thanks Barry, It solved the problem . if (bean == null) {//try with function because internal region does't have bean Set matchedMembers = new HashSet(); matchedMembers = CliUtil.getMembersForeRegionViaFunction(cache, region, true); return matchedMembers; } Can

RE: need information about SerialGatewaySenderQueue/ParallelGatewaySenderQueue Clear

2017-07-14 Thread Dinesh Akhand
Hi Team, Please reply . why we don't have implementation of clear method in ParallelGatewaySenderQueue/ SerialGatewaySenderQueue in geode. Requirement: we want to clear the queue data. I have implement below method in our code.