dengziming opened a new pull request #10247:
URL: https://github.com/apache/kafka/pull/10247


   *More detailed description of your change*
   Accidentally saw a log:
   ```
   [2021-03-02 20:39:56,418] DEBUG FindCoordinator request failed due to {} 
(org.apache.kafka.clients.consumer.internals.AbstractCoordinatorTest$DummyCoordinator:863)
   org.apache.kafka.common.errors.AuthenticationException: Authentication failed
   ```
   
   I fixed it to:
   ```
   [2021-03-02 20:51:39,050] ERROR FindCoordinator request failed due to 
Authentication failed 
(org.apache.kafka.clients.consumer.internals.AbstractCoordinatorTest$DummyCoordinator:863)
   ```
   
   I think the author wants to call `Logger.debug(String format, Object arg)`, 
but called `Logger.debug(String msg, Throwable t)`. In fact, there are 3 
overloadings `Logger.debug()` so we need to be careful.
   
   *Summary of testing strategy (including rationale)*
   test locally
   
   ### 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:
[email protected]


Reply via email to