diff --git a/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java b/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java index 0a849b3fa..803a49636 100644 --- a/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java +++ b/redisson/src/main/java/org/redisson/client/handler/PingConnectionHandler.java @@ -84,10 +84,7 @@ public class PingConnectionHandler extends ChannelInboundHandlerAdapter { Throwable cause = cause(future); - if (!(cause instanceof RedisLoadingException - || cause instanceof RedisTryAgainException - || cause instanceof RedisClusterDownException - || cause instanceof RedisBusyException)) { + if (!(cause instanceof RedisRetryException)) { if (!future.isCancelled()) { log.error("Unable to send PING command over channel: {}", ctx.channel(), cause); } diff --git a/redisson/src/main/java/org/redisson/command/RedisCommonBatchExecutor.java b/redisson/src/main/java/org/redisson/command/RedisCommonBatchExecutor.java index 794430dde..354aeddd4 100644 --- a/redisson/src/main/java/org/redisson/command/RedisCommonBatchExecutor.java +++ b/redisson/src/main/java/org/redisson/command/RedisCommonBatchExecutor.java @@ -161,10 +161,10 @@ public class RedisCommonBatchExecutor extends RedisExecutor { protected void handleResult(CompletableFuture attemptPromise, CompletableFuture connectionFuture) throws ReflectiveOperationException { if (attemptPromise.isDone() && !attemptPromise.isCompletedExceptionally()) { if (slots.decrementAndGet() == 0) { - mainPromise.complete(null); + handleSuccess(mainPromise, connectionFuture, null); } } else { - mainPromise.completeExceptionally(cause(attemptPromise)); + handleError(connectionFuture, cause(attemptPromise)); } }