From b7746c0732995ddfaf9f66d07baa3b15882b7ebf Mon Sep 17 00:00:00 2001 From: shenshichao163-oss Date: Sat, 15 Aug 2026 16:54:30 +0800 Subject: [PATCH] optimize: merge channel availability log output and adjust log level - Change info level from LOGGER.error() to LOGGER.info() (summary without stack traces) - Change debug level from LOGGER.error() to LOGGER.debug() (per-address detailed stack traces) - All failed addresses merged into one log entry at info level - Change else-if to independent if blocks so both levels work independently Fixes #5474 --- .../core/rpc/netty/NettyClientChannelManager.java | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/core/src/main/java/org/apache/seata/core/rpc/netty/NettyClientChannelManager.java b/core/src/main/java/org/apache/seata/core/rpc/netty/NettyClientChannelManager.java index dc3c4a15b3d..3f91e31f37d 100644 --- a/core/src/main/java/org/apache/seata/core/rpc/netty/NettyClientChannelManager.java +++ b/core/src/main/java/org/apache/seata/core/rpc/netty/NettyClientChannelManager.java @@ -251,17 +251,21 @@ void doReconnect(List availList, String transactionServiceGroup) { } } if (failedMap.size() > 0) { + // Merge all failed addresses into a single log entry to avoid flooding logs. + // info level: output summary without stack traces if (LOGGER.isInfoEnabled()) { - LOGGER.error( + LOGGER.info( "{} can not connect to {} cause:{}", FrameworkErrorCode.NetConnect.getErrCode(), failedMap.keySet(), failedMap.values().stream() .map(Throwable::getMessage) .collect(Collectors.toSet())); - } else if (LOGGER.isDebugEnabled()) { + } + // debug level: output detailed stack traces for each failed address + if (LOGGER.isDebugEnabled()) { failedMap.forEach((key, value) -> { - LOGGER.error( + LOGGER.debug( "{} can not connect to {} cause:{} trace information:", FrameworkErrorCode.NetConnect.getErrCode(), key,