showuon opened a new pull request #9627:
URL: https://github.com/apache/kafka/pull/9627


   This improvement is side effect of this PR: 
https://github.com/apache/kafka/pull/6972, where we tried to fix flooding 
number of warning messages but never left group case. So we move the warn logs 
into the `maybeLeaveGroup` method, and log as INFO level. 
(https://github.com/apache/kafka/pull/6972/files#diff-15efe9b844f78b686393b6c2e2ad61306c3473225742caed05c7edab9a138832L1092)
   
   The INFO log is good when the leaving group is expected, ex: unsubscribe 
from all topics, consumer close... However, there are some cases that are 
unexpected, ex: polling timeout, taking too long to read the log, and these 
cases should be logged as WARN because some Kafka users ignore INFO messages or 
have the log level set to WARN. 
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to