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

[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

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 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.,

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.,

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.,

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

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 Patrick Rhomberg
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60918/#review181273 ---

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

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:

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

[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,

[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

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.,

[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

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.,

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,

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

[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.

[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 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.,

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 ---

[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

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

[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

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

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

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

[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

[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

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]