Build failed in Jenkins: Geode-nightly #906

2017-07-24 Thread Apache Jenkins Server
See Changes: [jiliao] GEODE-393: GetRegionFunction uses the cache in the FunctionContext [jstewart] GEODE-3214: Remove support for multistep gfsh commands. [dbarnes] (no ticket) Update OSX to macOS [dschneider] GE

[GitHub] geode issue #535: GEODE-2986: Remove redundant log message

2017-07-24 Thread AkihiroKitada
Github user AkihiroKitada commented on the issue: https://github.com/apache/geode/pull/535 Hello folks, Thank you for paying your attention to this PR. --- 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 proj

[GitHub] geode pull request #577: Feature/geode 3049

2017-07-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/577 --- 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

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

2017-07-24 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/#review181283 --- Ship it! Ship It! - Darrel Schneider On July 21, 2017, 12:18

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #626 was SUCCESSFUL (with 1986 tests). Change made by John Blum.

2017-07-24 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #626 was successful. --- Scheduled with changes by John Blum. 1989 tests in total. https://build.spring.io/browse

Re: Review Request 61003: GEODE-393: GetRegionFunction uses the cache in the FunctionContext

2017-07-24 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61003/#review181275 --- Ship it! Ship It! - Jared Stewart On July 20, 2017, 5:40 p.m

Re: Review Request 60924: GEODE-2920: added security tests for create diskstore and create persistent region.

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60924/#review181274 --- Ship it! Ship It! - Emily Yeh On July 17, 2017, 9:39 p.m., J

[GitHub] geode pull request #654: GEODE-3291: add soundex analyzer into lucene query

2017-07-24 Thread gesterzhou
GitHub user gesterzhou opened a pull request: https://github.com/apache/geode/pull/654 GEODE-3291: add soundex analyzer into lucene query Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to ensure t

Re: Review Request 60918: GEODE-3134: fix lucene command security

2017-07-24 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60918/#review181273 --- geode-lucene/src/main/java/org/apache/geode/cache/lucene/internal

Re: Review Request 61003: GEODE-393: GetRegionFunction uses the cache in the FunctionContext

2017-07-24 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61003/#review181272 --- Ship it! Ship It! - Patrick Rhomberg On July 20, 2017, 5:40

Re: Review Request 60918: GEODE-3134: fix lucene command security

2017-07-24 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60918/#review181269 --- geode-core/src/test/java/org/apache/geode/test/dunit/rules/Member

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

2017-07-24 Thread Nick Reich
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60935/#review181270 --- Ship it! Ship It! - Nick Reich On July 21, 2017, 7:18 p.m.,

Re: Review Request 60918: GEODE-3134: fix lucene command security

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60918/#review181267 --- Ship it! Ship It! - Emily Yeh On July 17, 2017, 6:34 p.m., J

[GitHub] geode pull request #653: GEODE-3290: Removed effectively-dead classes Filter...

2017-07-24 Thread PurelyApplied
GitHub user PurelyApplied opened a pull request: https://github.com/apache/geode/pull/653 GEODE-3290: Removed effectively-dead classes FilterChain, et al Removed `FilterChain`, `LocalFilterChain`, and `RemoteFilterChain`. Minor cleanup of touched files. Precheckin r

Re: Review Request 61003: GEODE-393: GetRegionFunction uses the cache in the FunctionContext

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61003/#review181266 --- Ship it! Ship It! - Emily Yeh On July 20, 2017, 5:40 p.m., J

[GitHub] geode issue #652: Geode-2971: Introduce ExitCode to resolve inconsistency of...

2017-07-24 Thread PurelyApplied
Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/652 Precheckin running. --- 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

Re: Review Request 60977: GEODE-3251: make JMX test rules more robust

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60977/#review181264 --- Ship it! Ship It! - Emily Yeh On July 19, 2017, 5:14 p.m., K

[GitHub] geode pull request #652: Geode-2971: Introduce ExitCode to resolve inconsist...

2017-07-24 Thread PurelyApplied
GitHub user PurelyApplied opened a pull request: https://github.com/apache/geode/pull/652 Geode-2971: Introduce ExitCode to resolve inconsistency of shell exit values There are a few sections containing `// TODO PSRhomberg`. Most are there to mark a specific request for feedback, a

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

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60974/#review181263 --- Ship it! Ship It! - Emily Yeh On July 21, 2017, 8:05 p.m., J

Fixed: apache/geode#3249 (FunctionContext - cfa54f5)

2017-07-24 Thread Travis CI
Build Update for apache/geode - Build: #3249 Status: Fixed Duration: 9 minutes and 58 seconds Commit: cfa54f5 (FunctionContext) Author: Kirk Lund Message: Fixup View the changeset: https://github.com/apache/geode/compare/96e50e7e3073...cfa54f5ab17f View the

Failed: apache/geode#3248 (FunctionContext - 96e50e7)

2017-07-24 Thread Travis CI
Build Update for apache/geode - Build: #3248 Status: Failed Duration: 10 minutes and 16 seconds Commit: 96e50e7 (FunctionContext) Author: Kirk Lund Message: Convert to use FunctionContext.getCache() View the changeset: https://github.com/apache/geode/commit/96

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

2017-07-24 Thread Apache Jenkins Server
See Changes: [jiliao] GEODE-393: GetRegionFunction uses the cache in the FunctionContext -- [...truncated 98.28 KB...] Download https://repo1.maven.org/maven2/io/springf

Re: Review Request 60688: GEODE-1279: rename tests with old bug system numbers

2017-07-24 Thread Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60688/#review181256 --- Ship it! Ship It! - Patrick Rhomberg On July 6, 2017, 6:28 p

Re: Review Request 60688: GEODE-1279: rename tests with old bug system numbers

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60688/#review181251 --- Ship it! Ship It! - Emily Yeh On July 6, 2017, 6:28 p.m., Ki

Re: Review Request 61084: GEODE-3231: withLogFile does not imply withWorkingDir anymore

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61084/#review181249 --- Ship it! Ship It! - Emily Yeh On July 24, 2017, 4:28 p.m., J

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

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61042/#review181247 --- Ship it! +1! - Emily Yeh On July 24, 2017, 5:50 p.m., Jared

Re: Review Request 60666: geode-3166: remove the uncalled getCredential method

2017-07-24 Thread Emily Yeh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60666/#review181245 --- Ship it! +1! - Emily Yeh On July 17, 2017, 3:49 p.m., Jinmei

[GitHub] geode pull request #651: GEODE-3230: Cleaning up unused (Cli)Strings

2017-07-24 Thread YehEmily
GitHub user YehEmily opened a pull request: https://github.com/apache/geode/pull/651 GEODE-3230: Cleaning up unused (Cli)Strings [View the JIRA ticket here.](https://issues.apache.org/jira/browse/GEODE-3230) `CliStrings.java` contained a lot of Strings that were not used an

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

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

Re: Question regarding node failure scenario

2017-07-24 Thread Michael Stolz
Without subscription-redundancy you are running the risk that some of the data isn't being pushed to you in the event of a server failure. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Sun, Jul 23, 2017 at 10:06 PM, Akihiro Kitada wrote: > Hello Roi, > >

Re: Review Request 61084: GEODE-3231: withLogFile does not imply withWorkingDir anymore

2017-07-24 Thread Jared Stewart
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61084/#review181240 --- Ship it! Ship It! - Jared Stewart On July 24, 2017, 4:28 p.m

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

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

Build failed in Jenkins: Geode-nightly #905

2017-07-24 Thread Apache Jenkins Server
See -- [...truncated 132.79 KB...] :geode-json:build :geode-json:distributedTest UP-TO-DATE :geode-json:integrationTest UP-TO-DATE :geode-junit:compileTestJava :geode-junit:processTestResourc

[GitHub] geode-native pull request #111: Just a test PR

2017-07-24 Thread pivotal-jbarrett
GitHub user pivotal-jbarrett opened a pull request: https://github.com/apache/geode-native/pull/111 Just a test PR Please ignore this while we test PR CI You can merge this pull request into a Git repository by running: $ git pull https://github.com/pivotal-jbarrett/geode-nativ

Re: geode - stored procedures

2017-07-24 Thread Udo Kohlmeyer
Hi there Marios, Just to clarify, when you say "life oracle stored procedures up to the geode cluster", you mean that you are going to look at the functionality that each SP provided and code that up into a Geode Java Function code. As for best practices, have you looked at the following docu

Review Request 61084: GEODE-3231: withLogFile does not imply withWorkingDir anymore

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

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

2017-07-24 Thread echobravopapa
Github user echobravopapa closed the pull request at: https://github.com/apache/geode-native/pull/107 --- 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 fe

[GitHub] geode-native pull request #98: Test commit for Vince & Scott

2017-07-24 Thread echobravopapa
Github user echobravopapa closed the pull request at: https://github.com/apache/geode-native/pull/98 --- 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 fea

[GitHub] geode issue #535: GEODE-2986: Remove redundant log message

2017-07-24 Thread kohlmu-pivotal
Github user kohlmu-pivotal commented on the issue: https://github.com/apache/geode/pull/535 @AkihiroKitada we'll prioritize this merger. --- 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 f