You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@geode.apache.org by bs...@apache.org on 2017/02/15 21:48:19 UTC
geode git commit: Minor non-functional changes in response to PR
comments.
Repository: geode
Updated Branches:
refs/heads/feature/GEODE-2444 6d5b7a407 -> 15563faf1
Minor non-functional changes in response to PR comments.
Project: http://git-wip-us.apache.org/repos/asf/geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/geode/commit/15563faf
Tree: http://git-wip-us.apache.org/repos/asf/geode/tree/15563faf
Diff: http://git-wip-us.apache.org/repos/asf/geode/diff/15563faf
Branch: refs/heads/feature/GEODE-2444
Commit: 15563faf1327972e7452ab10ac4185c0d756fae9
Parents: 6d5b7a4
Author: Galen OSullivan <go...@pivotal.io>
Authored: Wed Feb 15 10:53:13 2017 -0800
Committer: Galen OSullivan <go...@pivotal.io>
Committed: Wed Feb 15 13:46:12 2017 -0800
----------------------------------------------------------------------
.../src/main/java/org/apache/geode/redis/GeodeRedisService.java | 3 ---
.../src/test/java/org/apache/geode/redis/AuthJUnitTest.java | 3 +--
2 files changed, 1 insertion(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/geode/blob/15563faf/geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java
----------------------------------------------------------------------
diff --git a/geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java b/geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java
index 0b32fe7..6e764ff 100644
--- a/geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java
+++ b/geode-core/src/main/java/org/apache/geode/redis/GeodeRedisService.java
@@ -16,9 +16,6 @@ package org.apache.geode.redis;
import org.apache.geode.internal.cache.CacheService;
-/**
- * Created by ukohlmeyer on 2/9/17.
- */
public interface GeodeRedisService extends CacheService {
/**
* The default Redis port as specified by their protocol, {@value #DEFAULT_REDIS_SERVER_PORT}
http://git-wip-us.apache.org/repos/asf/geode/blob/15563faf/geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java
----------------------------------------------------------------------
diff --git a/geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java b/geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java
index 2b03bd8..1aacf0a 100644
--- a/geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java
+++ b/geode-redis/src/test/java/org/apache/geode/redis/AuthJUnitTest.java
@@ -73,8 +73,7 @@ public class AuthJUnitTest extends RedisTestBase {
public void testAuthNoPwd() {
try (Jedis jedis = defaultJedisInstance()) {
jedis.auth(PASSWORD);
- fail(
- "We expecting either a JedisConnectionException or JedisDataException to be thrown here");
+ fail("Jedis auth to our Redis server should fail when we have no password set.");
} catch (JedisConnectionException | JedisDataException exception) {
}
}