[GitHub] geode issue #395: GEODE-2430: Fix failing tests

2017-02-07 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/geode/pull/395 +1 --- 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

[GitHub] geode issue #361: GEODE-2325: Improve tests for JarDeployer

2017-01-27 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/geode/pull/361 +1 --- 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

[GitHub] geode issue #327: GEODE-2236: Remove all cache-listners using Gfsh

2017-01-05 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/geode/pull/327 +1 --- 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

[GitHub] geode pull request #316: GEODE-2188: ExampleSecurityManager references Sampl...

2016-12-19 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/geode/pull/316#discussion_r93109555 --- Diff: geode-core/src/main/java/org/apache/geode/examples/security/ExampleSecurityManager.java --- @@ -257,15 +257,65 @@ private void readUsers(final Map

[GitHub] geode pull request #316: GEODE-2188: ExampleSecurityManager references Sampl...

2016-12-19 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/geode/pull/316#discussion_r93077824 --- Diff: geode-core/src/main/java/org/apache/geode/examples/security/ExampleSecurityManager.java --- @@ -257,15 +257,65 @@ private void readUsers(final Map

[GitHub] geode pull request #311: GEODE-2119: gfsh user and password in clear text

2016-12-12 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/geode/pull/311#discussion_r91999625 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/Gfsh.java --- @@ -14,38 +14,10 @@ */ --- End diff

[GitHub] incubator-geode issue #290: GEODE-2092: Security examples should not be in t...

2016-11-28 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/290 Merged in latest develop branch. 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

[GitHub] incubator-geode pull request #288: GEODE-2092 - Security examples should not...

2016-11-17 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/288 GEODE-2092 - Security examples should not be in the product code Moved examples to geode-examples project Copied old Sample/Simple code to test packages as many tests rely