github-forks
/
redisson
mirror of https://github.com/redisson/redisson.git
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

Fix the "of" string in RedisExecutor exceptions

Browse Source 
Signed-off-by: lkk7 <lukaszlapinski7@gmail.com>
pull/6465/head
Łukasz 4 weeks ago committed by lkk7
parent 5db527558a
commit 5c161775df
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File

12
redisson/src/main/java/org/redisson/command/RedisExecutor.java
Unescape Escape View File

@ -233,7 +233,7 @@ public class RedisExecutor<V, R> {
"Increase connection pool size or timeout. " "Increase connection pool size or timeout. "
+ "Node source: " + source + "Node source: " + source
+ ", " + LogHelper.toString(command, params) + ", " + LogHelper.toString(command, params)
+ " after " + attempt + "of " + attempts + " retry attempts"); + " after " + attempt + " of " + attempts + " retry attempts");
attemptPromise.completeExceptionally(exception); attemptPromise.completeExceptionally(exception);
} }
@ -258,7 +258,7 @@ public class RedisExecutor<V, R> {
+ " Netty pending tasks: " + pendingTasks + "," + " Netty pending tasks: " + pendingTasks + ","
+ " Node source: " + source + ", connection: " + connectionFuture.join() + " Node source: " + source + ", connection: " + connectionFuture.join()
+ ", " + LogHelper.toString(command, params) + ", " + LogHelper.toString(command, params)
+ " after " + attempt + "of " + attempts + " retry attempts"); + " after " + attempt + " of " + attempts + " retry attempts");
attemptPromise.completeExceptionally(exception); attemptPromise.completeExceptionally(exception);
} }
}; };
@ -284,7 +284,7 @@ public class RedisExecutor<V, R> {
"Increase connection pool size. " "Increase connection pool size. "
+ "Node source: " + source + "Node source: " + source
+ ", " + LogHelper.toString(command, params) + ", " + LogHelper.toString(command, params)
+ " after " + attempt + "of " + attempts + " retry attempts"); + " after " + attempt + " of " + attempts + " retry attempts");
} else { } else {
if (connectionFuture.isDone() && !connectionFuture.isCompletedExceptionally()) { if (connectionFuture.isDone() && !connectionFuture.isCompletedExceptionally()) {
if (writeFuture == null || !writeFuture.isDone()) { if (writeFuture == null || !writeFuture.isDone()) {
@ -298,7 +298,7 @@ public class RedisExecutor<V, R> {
" Netty pending tasks: " + pendingTasks + "," " Netty pending tasks: " + pendingTasks + ","
+ " Node source: " + source + ", connection: " + getNow(connectionFuture) + " Node source: " + source + ", connection: " + getNow(connectionFuture)
+ ", " + LogHelper.toString(command, params) + ", " + LogHelper.toString(command, params)
+ " after " + attempt + "of " + attempts + " retry attempts"); + " after " + attempt + " of " + attempts + " retry attempts");
} }
attemptPromise.completeExceptionally(exception); attemptPromise.completeExceptionally(exception);
} }
@ -376,7 +376,7 @@ public class RedisExecutor<V, R> {
"Node source: " "Node source: "
+ source + ", connection: " + connection + + source + ", connection: " + connection +
", " + LogHelper.toString(command, params) ", " + LogHelper.toString(command, params)
+ " after " + attempt + "of " + attempts + " retry attempts", + " after " + attempt + " of " + attempts + " retry attempts",
future.cause()); future.cause());
tryComplete(attemptPromise, exception); tryComplete(attemptPromise, exception);
return; return;
@ -465,7 +465,7 @@ public class RedisExecutor<V, R> {
int pendingTasks = countPendingTasks(); int pendingTasks = countPendingTasks();
attemptPromise.completeExceptionally( attemptPromise.completeExceptionally(
new RedisResponseTimeoutException("Redis server response timeout (" + timeoutAmount + " ms) occured" new RedisResponseTimeoutException("Redis server response timeout (" + timeoutAmount + " ms) occured"
+ " after " + attempt + "of " + attempts + " retry attempts," + " after " + attempt + " of " + attempts + " retry attempts,"
+ " is non-idempotent command: " + (command != null && command.isNoRetry()) + " is non-idempotent command: " + (command != null && command.isNoRetry())
+ " Check connection with Redis node: " + connection.getRedisClient().getAddr() + " for TCP packet drops or bandwidth limits. " + " Check connection with Redis node: " + connection.getRedisClient().getAddr() + " for TCP packet drops or bandwidth limits. "
+ " Try to increase nettyThreads and/or timeout settings." + " Try to increase nettyThreads and/or timeout settings."

Write Preview
Loading…
Cancel
Save