diff --git a/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisDirectPool.java b/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisPool.java similarity index 87% rename from modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisDirectPool.java rename to modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisPool.java index 8887ebb6..16931c13 100644 --- a/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisDirectPool.java +++ b/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisPool.java @@ -9,11 +9,11 @@ import redis.clients.util.Pool; /** * Pool which connect to redis instance directly. */ -public class JedisDirectPool extends Pool { +public class JedisPool extends Pool { private ConnectionInfo connectionInfo; - public JedisDirectPool(ConnectionInfo connectionInfo, JedisPoolConfig config) { + public JedisPool(ConnectionInfo connectionInfo, JedisPoolConfig config) { this.connectionInfo = connectionInfo; JedisFactory factory = new JedisFactory(connectionInfo.getHost(), connectionInfo.getPort(), diff --git a/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisSentinelPool.java b/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisSentinelPool.java index eaacca22..7c2ad593 100644 --- a/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisSentinelPool.java +++ b/modules/extension/src/main/java/org/springside/modules/nosql/redis/pool/JedisSentinelPool.java @@ -27,7 +27,7 @@ public class JedisSentinelPool extends Pool { private static Logger logger = LoggerFactory.getLogger(JedisSentinelPool.class); - private List sentinelPools = new ArrayList(); + private List sentinelPools = new ArrayList(); private MasterSwitchListener masterSwitchListener; private String masterName; @@ -57,7 +57,7 @@ public class JedisSentinelPool extends Pool { // check and assign parameter, all parameter can't not be null or empty assertArgument(((sentinelInfos != null) && (sentinelInfos.length != 0)), "seintinelInfos is not set"); for (ConnectionInfo sentinelInfo : sentinelInfos) { - JedisDirectPool sentinelPool = new JedisDirectPool(sentinelInfo, new JedisPoolConfig()); + JedisPool sentinelPool = new JedisPool(sentinelInfo, new JedisPoolConfig()); sentinelPools.add(sentinelPool); } @@ -77,9 +77,9 @@ public class JedisSentinelPool extends Pool { @Override public void destroy() { - masterSwitchListener.stopListening(); + masterSwitchListener.shutdown(); - for (JedisDirectPool sentinel : sentinelPools) { + for (JedisPool sentinel : sentinelPools) { sentinel.destroy(); } @@ -135,7 +135,7 @@ public class JedisSentinelPool extends Pool { public static final int RETRY_WAIT_TIME_MILLS = 1000; private JedisPubSub subscriber; - private JedisDirectPool currentSentinelPool; + private JedisPool currentSentinelPool; private Jedis subscriberSentinelJedis; private AtomicBoolean running = new AtomicBoolean(true); @@ -209,7 +209,7 @@ public class JedisSentinelPool extends Pool { /** * Interrupt the thread and stop the blocking subscription. */ - private void stopListening() { + private void shutdown() { running = new AtomicBoolean(false); this.interrupt(); @@ -227,8 +227,8 @@ public class JedisSentinelPool extends Pool { /** * Pickup the first available sentinel, if all sentinel down, return null. */ - private JedisDirectPool selectSentinel() { - for (JedisDirectPool sentinelPool : sentinelPools) { + private JedisPool selectSentinel() { + for (JedisPool sentinelPool : sentinelPools) { if (ping(sentinelPool)) { return sentinelPool; }