Re: PING message is "Mischief Managed"

+1 to the WarGames reference. On Fri, Jul 14, 2017 at 2:37 AM, Joey McAllister wrote: > + * for "pong." > > Alternatively: "Wouldn't you prefer a nice game of chess?" > > *[Unicode character U+1F3D3] > > On Thu, Jul 13, 2017 at 1:54 PM Michael William Dodge

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/#review180500 --- Ship it! Ship It! - Brian Rowe On July 13, 2017, 11:28

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/#review180497 --- Fix the logging statement & you're good to go

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60856/ --- Review request for geode, Alexander Murmann, Bruce Schuchardt, Galen

[GitHub] geode pull request #634: Feature/geode 3175

GitHub user bschuchardt opened a pull request: https://github.com/apache/geode/pull/634 Feature/geode 3175 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

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

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

No nulls

Please try to imagine a world of no nulls. https://www.oracle.com/corporate/features/library-in-java-best-practices.html?evite=WWMK170414P4

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

GitHub user galen-pivotal opened a pull request: https://github.com/apache/geode/pull/633 GEODE-3170: Closed socket doesn't result in an infinite loop. * Protobuf deserialization returning null is handled. * IOException causes GenericProtocolServerConnection to close. *

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

GitHub user WireBaron opened a pull request: https://github.com/apache/geode/pull/632 GEODE-3203: fixing protobuf build.gradle to respect buildRoot Signed-off-by: Bruce Schuchardt Thank you for submitting a contribution to Apache Geode. In

Re: Proposal: Lucene indexing/searching for nested objects

On Thu, Jul 13, 2017 at 11:26 AM, Jacob Barrett wrote: > Collections are really tough in Lucene because you have to flatten the > document. I struggled against it for some time on a project a few years ago > and ultimately decided to index the relationships separately and

Re: PING message is "Mischief Managed"

+ * for "pong." Alternatively: "Wouldn't you prefer a nice game of chess?" *[Unicode character U+1F3D3] On Thu, Jul 13, 2017 at 1:54 PM Michael William Dodge wrote: > +1 for "pong". > > > On 13 Jul, 2017, at 13:45, Kirk Lund wrote: > > > > I'm really

RE: Stored procedures on Apache Geode.

Hi John, Thank you so much for your advise. I will go through the links you provided and get back to you if I have a question. I really appreciate it! Marios From: John Blum [via Apache Geode (Incubating) Developers Forum] [ml+s70738n24325...@n6.nabble.com] Sent:

Re: PING message is "Mischief Managed"

Of course, *ping()* could always return... *"you've got to ask yourself one question: "Do I feel lucky?" Well, do ya, punk?"* ;-) https://en.wikiquote.org/wiki/Dirty_Harry On Thu, Jul 13, 2017 at 1:54 PM, Michael William Dodge wrote: > +1 for "pong". > > > On 13 Jul, 2017,

Re: PING message is "Mischief Managed"

+1 for "pong". > On 13 Jul, 2017, at 13:45, Kirk Lund wrote: > > I'm really boring... I would've just had it reply "PONG" ;) > > On Thu, Jul 13, 2017 at 11:57 AM, John Blum wrote: > >> Here you go... >> >>

Re: PING message is "Mischief Managed"

LOL, I thought about that too. Actually, most of my ping service endpoints I write now (whether HTTP or other... TCP/IP) just respond with "PONG", as you said. I don't think that is boring at all. On Thu, Jul 13, 2017 at 1:45 PM, Kirk Lund wrote: > I'm really boring... I

Re: PING message is "Mischief Managed"

I'm really boring... I would've just had it reply "PONG" ;) On Thu, Jul 13, 2017 at 11:57 AM, John Blum wrote: > Here you go... > > https://github.com/apache/geode/blob/rel/v1.1.1/geode- > core/src/main/java/org/apache/geode/management/internal/web/shell/ >

[GitHub] geode-examples pull request #10: Feature/geode 3202

GitHub user dihardman opened a pull request: https://github.com/apache/geode-examples/pull/10 Feature/geode 3202 Here is a new simple example that uses Lucene indexes and searching. Please review and comment. You can merge this pull request into a Git repository by running: $

RE: Stored procedures on Apache Geode.

From: John Blum [via Apache Geode (Incubating) Developers Forum] [ml+s70738n24312...@n6.nabble.com] Sent: Thursday, July 13, 2017 9:16 PM To: Marios Sofocleous/IT/CREDITSAFE Subject: Re: Stored procedures on Apache Geode. Right. You can also review the

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127312259 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -78,13 +78,12 @@ int32_t StructSetImpl::getFieldIndex(const char* fieldname) { }

Re: PING message is "Mischief Managed"

Here you go... https://github.com/apache/geode/blob/rel/v1.1.1/geode-core/src/main/java/org/apache/geode/management/internal/web/shell/RestHttpOperationInvoker.java#L151-L204 On Thu, Jul 13, 2017 at 10:51 AM, John Blum wrote: > Corrections below (apologies)... > > On Thu,

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

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127299289 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/PutAllRequestOperationHandlerJUnitTest.java --- @@ -14,6 +14,14 @@

Re: Proposal: Lucene indexing/searching for nested objects

Collections are really tough in Lucene because you have to flatten the document. I struggled against it for some time on a project a few years ago and ultimately decided to index the relationships separately and then merge the results. Sent from my iPhone > On Jul 13, 2017, at 11:13 AM, Dan

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

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127294137 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/RoundTripCacheConnectionJUnitTest.java --- @@ -278,6 +278,31 @@ public void

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

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127294096 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufUtilities.java --- @@ -133,21 +134,38 @@ /**

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

+1 based on following sanity tests - started gfsh , created locator, server , region and executed few basic queries. - Using the libs in a project , created a server , filled it with objects , executed DISTINCT, ORDER BY queries. - Executed "./gradlew clean build

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

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127292855 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/protobuf/operations/PutAllRequestOperationHandlerJUnitTest.java --- @@ -14,6 +14,14 @@

Build failed in Jenkins: Geode-nightly #895

See Changes: [boglesby] GEODE-3181: Added wan backward compatibility unit tests [eshu] GEODE-3178: Fix a flaky JTA test by adding retry. [dschneider] GEODE-3194: cleanup disk store on failed initial recovery

Re: Proposal: Lucene indexing/searching for nested objects

+1 Looks good. I think we should consider adding support for collections as well, but that doesn't have to be in the first cut. -Dan On Wed, Jul 12, 2017 at 10:37 AM, Diane Hardman wrote: > The Geode 1.2.0 release includes Lucene text search fully integrated and > tested

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

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127290974 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/RoundTripCacheConnectionJUnitTest.java --- @@ -278,6 +278,31 @@ public void

Re: Apache Geode + Narayana JTA = Global Transactions with Last-Resource Optimization

Hi Christian- Certainly, if you like, I would be happy to help you out with the new Repo, where I can. @Geode Community- As an FYI, I also just completed DATAGEODE-16 [1] for *Spring Data Geode* (as well as SGF-636 [2] for *Spring Data GemFire*) which includes support for Apache Geode in

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

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127289532 --- Diff: geode-protobuf/src/main/proto/basicTypes.proto --- @@ -52,7 +52,12 @@ message CallbackArguments { message Region { string

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

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127289486 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/RoundTripCacheConnectionJUnitTest.java --- @@ -278,6 +278,31 @@ public void

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

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127289396 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/RemoveRequestOperationHandler.java --- @@ -33,7 +33,7 @@

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user echobravopapa commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127285583 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -66,8 +66,8 @@ SelectResultsIterator StructSetImpl::getIterator() { return

Re: PING message is "Mischief Managed"

Corrections below (apologies)... On Thu, Jul 13, 2017 at 10:44 AM, John Blum wrote: > Yeah, that was my doing. :) > > I thought it would be more fun to return a special message than the > typical "Alive". "Mischief Managed" comes from *Harry Potter*. > > This endpoint is of

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user echobravopapa commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127285563 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -78,13 +78,12 @@ int32_t StructSetImpl::getFieldIndex(const char* fieldname) { }

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

Github user WireBaron commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127285451 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/operations/RemoveRequestOperationHandler.java --- @@ -33,7 +33,7 @@

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

Github user pivotal-amurmann commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127285154 --- Diff: geode-protobuf/src/test/java/org/apache/geode/protocol/RoundTripCacheConnectionJUnitTest.java --- @@ -278,6 +278,31 @@ public void

Re: PING message is "Mischief Managed"

Yeah, that is my doing. :) I thought it would be more fun to return a special message than the typical "Alive". "Mischief Managed" comes from Harry Potter. This endpoint is of course benign/idempotent and was purely meant to test the Management REST API's availability, or rather that the

Re: Review Request 60523: GEODE-3141 New flow: GetRegion

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

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

Github user bschuchardt commented on a diff in the pull request: https://github.com/apache/geode/pull/630#discussion_r127276853 --- Diff: geode-protobuf/src/main/proto/basicTypes.proto --- @@ -52,7 +52,12 @@ message CallbackArguments { message Region { string

Re: PING message is "Mischief Managed"

For the older crowed I would have rather it replied: Ah, I see you have the machine that goes 'ping!'. Sent from my iPhone > On Jul 13, 2017, at 10:23 AM, Jared Stewart wrote: > > I'm young enough to recognize it as a Harry Potter reference, but I have no > idea what it's

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127278256 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -66,8 +66,8 @@ SelectResultsIterator StructSetImpl::getIterator() { return

Re: PING message is "Mischief Managed"

I'm young enough to recognize it as a Harry Potter reference, but I have no idea what it's doing in our product code. - Jared On Jul 13, 2017 10:14 AM, "Kirk Lund" wrote: > Anyone know why the response to a REST service PING returns "Mischief > Managed!? > >

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127278443 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -66,8 +66,8 @@ SelectResultsIterator StructSetImpl::getIterator() { return

[GitHub] geode-native pull request #107: GEODE-3019: Refactor Struct class

Github user pivotal-jbarrett commented on a diff in the pull request: https://github.com/apache/geode-native/pull/107#discussion_r127278631 --- Diff: src/cppcache/src/StructSetImpl.cpp --- @@ -78,13 +78,12 @@ int32_t StructSetImpl::getFieldIndex(const char* fieldname) { }

PING message is "Mischief Managed"

Anyone know why the response to a REST service PING returns "Mischief Managed!? @RequestMapping(method = {RequestMethod.GET, RequestMethod.HEAD}, value = "/ping") public ResponseEntity ping() { return new ResponseEntity("Mischief Managed!", HttpStatus.OK); }

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60834/#review180441 --- Ship it! Ship It! - Bruce Schuchardt On July 12, 2017, 5:35

[GitHub] geode-native issue #107: GEODE-3019: Refactor Struct class

Github user echobravopapa commented on the issue: https://github.com/apache/geode-native/pull/107 @dgkimura @pivotal-jbarrett added unit tests and throwing an exception for out of range now --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] geode issue #631: GEODE-3051: Remove unreachable exception handling in Accep...

Github user WireBaron commented on the issue: https://github.com/apache/geode/pull/631 Isn't it the same code handling the SSL handshake for both protocols right now? If it's a different code path, then we should probably add a new test for it. On Thu, Jul 13, 2017 at

Re: Geode-Native Windows build

Yup! It should match the the minimum requirements mentioned at the very least. It should also compile fine under 2017 though I don't believe anyone has yet. Sent from my iPhone > On Jul 13, 2017, at 6:08 AM, Daniel Farcovich > wrote: > > From BUILDING.md: > >

RE: Geode-Native Windows build

>From BUILDING.md: Windows Generator The recommended generator on Windows is Visual Studio 12 2013 Win64: $ cmake -G "Visual Studio 12 2013 Win64" ../src Should we change that? Daniel -Original Message- From: Jacob Barrett [mailto:jbarr...@pivotal.io] Sent: Thursday, July 13, 2017

Re: Stored procedures on Apache Geode.

A geode Function is conceptually similar to a stored proc but the implementation is different. It's a java class that would use geode apis to execute logic within the cluster. Anthony > On Jul 12, 2017, at 11:13 PM, marios390 > wrote: > > Hi, > > I

Re: Geode-Native Windows build

VS 2013 is not fully C++11 compliant. The minimum compiler on windows is VS 2015. Sent from my iPhone > On Jul 13, 2017, at 4:32 AM, Daniel Farcovich > wrote: > > As I mentioned in the first email I receive compilation errors, for example: >

RE: Geode-Native Windows build

As I mentioned in the first email I receive compilation errors, for example: 31>C:\Users\Fdaniel\Source\Repos\geode-native2\src\cppcache\include\geode/utils.hpp(57): error C2144: syntax error : 'bool' should be preceded by ';'

Re: Apache Geode + Narayana JTA = Global Transactions with Last-Resource Optimization

Hi As suggested i've added related wiki page here: https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=71764563# Feel free to review and edit! @Swapnil, @John, since this is an external, optional utility it might useful to add you as github (

RE: Geode-Native Windows build

Hi Jake and Ernest, I tried to build from c:\build Here is the config command and output: c:\build>cmake -DCMAKE_INSTALL_PREFIX=C:\build\out -G "Visual Studio 12 2013 Win64" -Thost=x64 C:\Users\Fdaniel\Source\Repos\geode-native2\src -- The C compiler identification is MSVC 18.0.21005.1 -- The

Stored procedures on Apache Geode.

Hi, I noticed that Apache Geode supports stored procedures as functions running on nodes. Could you please elaborate on that since I am confused a little a bit. Can I run for example Oracle SP directly on cache servers (nodes) or do I have to transform them somehow into microservices ( i.e spring