geode - stored procedures

Hi Markito, Here in Creditsafe we are planning to lift oracles stored procedures up to the geode cluster running as Functions. Could you please assists us by providing best practices , or any demo related to this. If you are not aware of this particular application , I would appreciate it you c

Build failed in Jenkins: Geode-nightly-flaky #72

See Changes: [jstewart] GEODE-3217: Reimplement gfsh query as a single-step command [jiliao] GEODE-393: GetRegionFunction uses the cache in the FunctionContext [jiliao] GEODE-3231: do not use temporary folder

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

See

[GitHub] geode pull request #506: GEODE-2900: DUnit test of moving primary during AEQ...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/506 --- 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 enabl

[GitHub] geode pull request #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/646 --- 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 enabl

[GitHub] geode issue #647: GEODE-3271: Refactor WanCommands

Github user YehEmily commented on the issue: https://github.com/apache/geode/pull/647 @PurelyApplied Updated! Thanks for the feedback! --- 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 fea

[GitHub] geode pull request #648: GEODE-3206 : Corrected order of lucene search resul...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/648 --- 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 enabl

[GitHub] geode issue #643: GEODE-3192,GEODE-3229: Change API and implementation of pr...

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/643 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

[GitHub] geode issue #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/646 If you are using IntelliJ as your IDE, please remember to update your style file to that located in `geode/etc/intellij-java-modified-google-style.xml` to be consistent with Geode's established

[GitHub] geode issue #647: GEODE-3271: Refactor WanCommands

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/647 +1 as it stands. Unimportant nitpicks, rambling observations, and "it could be even better if...": * `punePort` appears in these tests a lot. I have no idea what `pune` is sup

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

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

Re: Geode PR pile up

I'm merging PR 648 now On Fri, Jul 21, 2017 at 3:32 PM, Jacob Barrett wrote: > All, > > I followed up on the issue regarding abandoned PRs. The only way to close > them if the user has walked away is to do an empty commit. Thanks to Mark > for finding this ticket with explanation > https://issue

Re: Geode PR pile up

All, I followed up on the issue regarding abandoned PRs. The only way to close them if the user has walked away is to do an empty commit. Thanks to Mark for finding this ticket with explanation https://issues.apache.org/jira/browse/INFRA-13690 and instructions on how to do it. I would say any PR

Fixed: apache/geode#3235 (develop - 56d964f)

Build Update for apache/geode - Build: #3235 Status: Fixed Duration: 18 minutes and 49 seconds Commit: 56d964f (develop) Author: Jinmei Liao Message: GEODE-393: revert - GetRegionFunction uses the cache in the FunctionContext View the changeset: https://gith

Broken: apache/geode#3233 (develop - 0ea489e)

Build Update for apache/geode - Build: #3233 Status: Broken Duration: 10 minutes and 25 seconds Commit: 0ea489e (develop) Author: Jinmei Liao Message: GEODE-3231: do not use temporary folder as workingdir or log to file by default when using the member starter

Broken: apache/geode#3232 (develop - 5551b3f)

Build Update for apache/geode - Build: #3232 Status: Broken Duration: 11 minutes and 58 seconds Commit: 5551b3f (develop) Author: Jinmei Liao Message: GEODE-393: GetRegionFunction uses the cache in the FunctionContext View the changeset: https://github.com/ap

Re: Review Request 60974: GEODE-3231: do not log to file by default when using the member starter rules

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60974/ --- (Updated July 21, 2017, 8:05 p.m.) Review request for geode, Emily Yeh, Jared S

[GitHub] geode pull request #649: GEODE-2997: Change new protocol GetAllResponse.

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/649#discussion_r128848930 --- Diff: geode-protobuf/src/main/proto/basicTypes.proto --- @@ -62,4 +62,14 @@ message Region { message Server { string url = 1;

[GitHub] geode pull request #649: GEODE-2997: Change new protocol GetAllResponse.

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/649#discussion_r128847166 --- Diff: geode-protobuf/src/main/proto/clientProtocol.proto --- @@ -66,7 +66,7 @@ message Response { RemoveAllResponse removeAllResponse =

Re: Review Request 60935: GEODE-3115 Added changes to check for persistent region during pdx type registry.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/ --- (Updated July 21, 2017, 7:18 p.m.) Review request for geode, Darrel Schneider,

[GitHub] geode issue #580: GEODE-2936: Refactoring OrderByComparator

Github user YehEmily commented on the issue: https://github.com/apache/geode/pull/580 @jhuynh1 Thanks for the suggestion! I updated the tests in `OrderByComparatorJUnitTest.java` to cover all the cases in `OrderByComparator.java` and updated the PR. --- If your project is set up for

Re: Review Request 61042: GEODE-3214: Remove support for multistep gfsh commands.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61042/#review181133 --- Ship it! Ship It! - Jinmei Liao On July 21, 2017, 5:36 p.m.,

[GitHub] geode pull request #650: GEODE-3253: Refactoring ClientCommands and related ...

GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/650 GEODE-3253: Refactoring ClientCommands and related tests [View the JIRA ticket here.](https://issues.apache.org/jira/browse/GEODE-3253) `ClientCommands.java` was a large class that containe

Review Request 61042: GEODE-3214: Remove support for multistep gfsh commands.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61042/ --- Review request for geode, Emily Yeh, Jared Stewart, Ken Howe, Kirk Lund, and Pat

Jenkins build is back to normal : Geode-release-flaky #28

See

[GitHub] geode pull request #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user kohlmu-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/646#discussion_r128814031 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/OperationContext.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the A

Re: Review Request 60974: GEODE-3231: do not log to file by default when using the member starter rules

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60974/ --- (Updated July 21, 2017, 5:11 p.m.) Review request for geode, Emily Yeh, Jared S

[GitHub] geode pull request #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/646#discussion_r128811946 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/OperationContext.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the Ap

[GitHub] geode pull request #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user galen-pivotal commented on a diff in the pull request: https://github.com/apache/geode/pull/646#discussion_r128811551 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufResponseUtilities.java --- @@ -35,10 +37,9 @@ * @pa

Re: Avoid mutating standard Java System Properties (even in tests)

There is no issue with setting GemFire-specific System Properties. Feel free to use those. I'm talking about changing the values of System Properties set by the JVM such as "user.dir" -- mutating or clearing these values can cause weird bugs in the JDK or 3rd party libraries that are harder to debu

Re: Avoid mutating standard Java System Properties (even in tests)

Come on guys, it's 2017 and we should not still be talking about using system properties at all. I won't rant again about our prolific use of system properties but I will rant on the testing using them. Unless your test is a unit test that is asserting that your component picks up its initial state

[GitHub] geode pull request #646: GEODE-3213: Refactor ProtoBuf handler flow

Github user pivotal-amurmann commented on a diff in the pull request: https://github.com/apache/geode/pull/646#discussion_r128793895 --- Diff: geode-protobuf/src/main/java/org/apache/geode/protocol/protobuf/utilities/ProtobufResponseUtilities.java --- @@ -50,47 +38,14 @@ *

Build failed in Jenkins: Geode-release #79

See -- [...truncated 1.19 MB...] at org.jgroups.JChannel.connect(JChannel.java:288) at org.jgroups.JChannel.connect(JChannel.java:279) at org.apache.geode.distributed.