[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_r93079596 --- 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 #316: GEODE-2188: ExampleSecurityManager references Sampl...

2016-12-15 Thread kjduling
Github user kjduling commented on a diff in the pull request: https://github.com/apache/geode/pull/316#discussion_r92662419 --- 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-14 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/geode/pull/316 GEODE-2188: ExampleSecurityManager references SampleSecurityManager i… …n javadoc This corrects some output messages and javadoc entries after the ExampleSecurityManager was moved. It

[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] geode pull request #311: GEODE-2119: gfsh user and password in clear text

2016-12-09 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/geode/pull/311 GEODE-2119: gfsh user and password in clear text You can merge this pull request into a Git repository by running: $ git pull https://github.com/kjduling/apache-geode feature/GEODE-2119

[GitHub] incubator-geode pull request #302: GEODE-2196: Subclasses of OperationContex...

2016-12-02 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/302 GEODE-2196: Subclasses of OperationContext should also be deprecated. precheckin running You can merge this pull request into a Git repository by running: $ git pull https

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

2016-11-29 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/290 @metatype Yes, precheckin ran clean. --- 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

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

[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 #290: GEODE-2092: Security examples should not ...

2016-11-18 Thread kjduling
GitHub user kjduling opened a pull request: https://github.com/apache/incubator-geode/pull/290 GEODE-2092: Security examples should not be in the product code Moved examples to geode-core/src/main/java/org/apache/geode/examples/security/ Copied old Sample/Simple code to test

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

2016-11-18 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/288 Cancelling this one to resubmit as a single, clean commit. --- 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

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

2016-11-18 Thread kjduling
Github user kjduling closed the pull request at: https://github.com/apache/incubator-geode/pull/288 --- 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

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

2016-11-17 Thread kjduling
Github user kjduling commented on the issue: https://github.com/apache/incubator-geode/pull/288 I moved the examples in to geode-examples/utils/src/main/java/org/apache/geode/example/security/ so it looks like I misunderstood and jumped the gun. So, I'll undo that, and move th

[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 on these