kirktrue commented on code in PR #16345: URL: https://github.com/apache/kafka/pull/16345#discussion_r1643453077
########## clients/src/main/java/org/apache/kafka/clients/consumer/internals/MembershipManagerImpl.java: ########## @@ -362,7 +362,7 @@ private void transitionTo(MemberState nextState) { metricsManager.recordRebalanceStarted(time.milliseconds()); } - log.trace("Member {} with epoch {} transitioned from {} to {}.", memberId, memberEpoch, state, nextState); + log.info("Member {} with epoch {} transitioned from {} to {}.", memberId, memberEpoch, state, nextState); Review Comment: OK, yes, that totally makes sense 👍 Plus, when community members file issues against KIP-848 client work, it's better to have those logs included by default. Good call 😄 -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org