From 69d879f72642c80f89763303ac256b1e70ccf26b Mon Sep 17 00:00:00 2001 From: Nikita Koksharov Date: Tue, 4 Dec 2018 20:18:07 +0300 Subject: [PATCH] Fixed - Config node address should be filled out. --- .../java/org/redisson/cluster/ClusterConnectionManager.java | 4 ++++ .../org/redisson/connection/SentinelConnectionManager.java | 3 +++ 2 files changed, 7 insertions(+) diff --git a/redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java b/redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java index fb5dc9f40..13ba7b83b 100644 --- a/redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java +++ b/redisson/src/main/java/org/redisson/cluster/ClusterConnectionManager.java @@ -92,6 +92,10 @@ public class ClusterConnectionManager extends MasterSlaveConnectionManager { public ClusterConnectionManager(ClusterServersConfig cfg, Config config, UUID id) { super(config, id); + if (cfg.getNodeAddresses().isEmpty()) { + throw new IllegalArgumentException("At least one cluster node should be defined!"); + } + this.config = create(cfg); initTimer(this.config); diff --git a/redisson/src/main/java/org/redisson/connection/SentinelConnectionManager.java b/redisson/src/main/java/org/redisson/connection/SentinelConnectionManager.java index 67a668094..0a4a72f7c 100755 --- a/redisson/src/main/java/org/redisson/connection/SentinelConnectionManager.java +++ b/redisson/src/main/java/org/redisson/connection/SentinelConnectionManager.java @@ -78,6 +78,9 @@ public class SentinelConnectionManager extends MasterSlaveConnectionManager { if (cfg.getMasterName() == null) { throw new IllegalArgumentException("masterName parameter is not defined!"); } + if (cfg.getSentinelAddresses().isEmpty()) { + throw new IllegalArgumentException("At least one sentinel node should be defined!"); + } this.config = create(cfg); initTimer(this.config);