Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

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

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/#review178075 --- Thank you Jens! This is awesome. - Jared Stewart On June 16,

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/#review178070 --- So much awesome. - Anthony Baker On June 16, 2017, 12:59 a.m.,

Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60142/ --- Review request for geode, Anthony Baker, Mark Bretl, and Dan Smith.

[GitHub] geode issue #584: GEODE-2901: Adding integration tests of session replicatio...

Github user DivineEnder commented on the issue: https://github.com/apache/geode/pull/584 @ladyVader @nabarunnag @boglesby @jhuynh1 @upthewaterspout @gesterzhou --- 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 #584: GEODE-2901: Adding integration tests of session rep...

GitHub user DivineEnder opened a pull request: https://github.com/apache/geode/pull/584 GEODE-2901: Adding integration tests of session replication Adding integration tests that test both session replications frameworks shipped with geode. These tests use cargo to download

Re: Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

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

[GitHub] geode pull request #582: GEODE-2601: Fix banner being logged twice

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/582#discussion_r122332971 --- Diff: geode-core/src/main/java/org/apache/geode/distributed/internal/InternalLocator.java --- @@ -469,15 +470,13 @@ private InternalLocator(int port,

[GitHub] geode pull request #582: GEODE-2601: Fix banner being logged twice

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/582#discussion_r122332566 --- Diff: geode-core/src/main/java/org/apache/geode/distributed/internal/InternalDistributedSystem.java --- @@ -2167,7 +2191,7 @@ private static void

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

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

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

Github user karensmolermiller commented on a diff in the pull request: https://github.com/apache/geode/pull/583#discussion_r122326144 --- Diff: geode-docs/rest_apps/setup_config.html.md.erb --- @@ -29,162 +31,233 @@ All Geode REST interface classes and required JAR files are

Review Request 60125: GEODE-3062: replace SecurityService after loading cluster config [FIXED]

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60125/ --- Review request for geode, Emily Yeh, Jinmei Liao, Ken Howe, and Patrick

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

Github user joeymcallister commented on a diff in the pull request: https://github.com/apache/geode/pull/583#discussion_r122319962 --- Diff: geode-docs/rest_apps/setup_config.html.md.erb --- @@ -29,162 +31,233 @@ All Geode REST interface classes and required JAR files are

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

Github user joeymcallister commented on a diff in the pull request: https://github.com/apache/geode/pull/583#discussion_r122320206 --- Diff: geode-docs/rest_apps/setup_config.html.md.erb --- @@ -29,162 +31,233 @@ All Geode REST interface classes and required JAR files are

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

Github user joeymcallister commented on a diff in the pull request: https://github.com/apache/geode/pull/583#discussion_r122320074 --- Diff: geode-docs/rest_apps/setup_config.html.md.erb --- @@ -29,162 +31,233 @@ All Geode REST interface classes and required JAR files are

[GitHub] geode pull request #583: GEODE-2854: GEODE REST API Docs Missing Authenticat...

GitHub user davebarnes97 opened a pull request: https://github.com/apache/geode/pull/583 GEODE-2854: GEODE REST API Docs Missing Authentication Reviews requested from: @mmartell, @jaredjstewart @karensmolermiller @joeymcallister You can merge this pull request into a Git

[GitHub] geode pull request #578: GEODE-1958: Removing/deprecating PasswordUtil

Github user YehEmily commented on a diff in the pull request: https://github.com/apache/geode/pull/578#discussion_r122279107 --- Diff: geode-core/src/test/java/org/apache/geode/cache/util/PasswordUtilJUnitTest.java --- @@ -1,42 +0,0 @@ -/* - * Licensed to the Apache

[GitHub] geode pull request #581: GEODE-2301 Doc note to deprecate Geode JTA trans mg...

GitHub user karensmolermiller opened a pull request: https://github.com/apache/geode/pull/581 GEODE-2301 Doc note to deprecate Geode JTA trans mgr Simple addition to the beginning of the documentation of the Geode JTA transaction manager that the Geode JTA transaction manager is

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

See