aloyszhang commented on a change in pull request #12823:
URL: https://github.com/apache/pulsar/pull/12823#discussion_r750393745
##########
File path:
pulsar-client/src/main/java/org/apache/pulsar/client/impl/ClientCnx.java
##########
@@ -253,7 +253,7 @@ protected ByteBuf newConnectCommand() throws Exception {
@Override
public void channelInactive(ChannelHandlerContext ctx) throws Exception {
super.channelInactive(ctx);
- log.info("{} Disconnected", ctx.channel());
+ log.warn("{} Disconnected", ctx.channel());
Review comment:
As described in #12755, if the channel becomes an inactive state may be
caused by topic unload or broker reboot. If we only have INFO log, it's very
easy to miss the disconnection information since there're many INFO logs.
So I think with a warn log, it's helpful for users to perceive the change
and understand the client traffic undulate.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
