You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by GitBox <gi...@apache.org> on 2020/05/28 18:24:45 UTC

[GitHub] [kafka] vvcephei opened a new pull request #8747: MINOR: Log the reason for coordinator discovery failure

vvcephei opened a new pull request #8747:
URL: https://github.com/apache/kafka/pull/8747


   Adds the exception in the case of logging the "Coordinator discovery failed" message.
   
   ### 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:
users@infra.apache.org

[GitHub] [kafka] vvcephei commented on pull request #8747: MINOR: Log the reason for coordinator discovery failure

Posted by GitBox <gi...@apache.org>.
vvcephei commented on pull request #8747:
URL: https://github.com/apache/kafka/pull/8747#issuecomment-635521481


   Hey @hachikuji , Can you take a quick look at this? You were the last to edit this block.
   
   I was debugging a system test failure, and I see a situation where the broker appears completely healthy, but the Streams consumer just prints `Coordinator discovery failed, refreshing metadata` for several minutes before the test times out.
   
   I didn't see any way to get more information on what exactly the failure was, so I'd like to add the actual causing exception to the log line. Since it's a debug log, this _seems_ ok, but I'm interested in feedback.
   
   Thanks!


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [kafka] vvcephei commented on pull request #8747: MINOR: Log the reason for coordinator discovery failure

Posted by GitBox <gi...@apache.org>.
vvcephei commented on pull request #8747:
URL: https://github.com/apache/kafka/pull/8747#issuecomment-635659313


   Thanks for the review, @abbccdda !


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [kafka] vvcephei merged pull request #8747: MINOR: Log the reason for coordinator discovery failure

Posted by GitBox <gi...@apache.org>.
vvcephei merged pull request #8747:
URL: https://github.com/apache/kafka/pull/8747


   


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [kafka] vvcephei commented on pull request #8747: MINOR: Log the reason for coordinator discovery failure

Posted by GitBox <gi...@apache.org>.
vvcephei commented on pull request #8747:
URL: https://github.com/apache/kafka/pull/8747#issuecomment-635602993


   The java 14 test failure is unrelated and fixed in https://github.com/apache/kafka/pull/8748


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [kafka] vvcephei commented on pull request #8747: MINOR: Log the reason for coordinator discovery failure

Posted by GitBox <gi...@apache.org>.
vvcephei commented on pull request #8747:
URL: https://github.com/apache/kafka/pull/8747#issuecomment-635659034


   Java 11 build timed out.


----------------------------------------------------------------
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:
users@infra.apache.org