Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sometimes KafkaCdiExtensionTest fails #1816

Merged
merged 1 commit into from
May 19, 2020
Merged

Conversation

jbescos
Copy link
Member

@jbescos jbescos commented May 18, 2020

#1735

First fix didn't make it work. I am still not able to reproduce it locally, but I did some improvements based on the logs I saw here https://builds.helidon.io/api/DA2B3917F7CA946F13C9B631B711C66C/artifact/13/tests/integration/kafka/target/surefire-reports/io.helidon.messaging.connectors.kafka.KafkaCdiExtensionTest-output.txt

  • Tests will make sure they commit messages before they finish.
  • Tests will wait till all the messages are published. With this we will know if the problem was producing the message, or consuming it.
  • There is one test that is publishing in a topic that doesn't exist. Some internal threads of Kafka write some warn lines again and again. Changing some Kafka properties doesn't help with that, so the test will create the topic at the end to avoid this issue.
  • We log more, so next time maybe I will have more relevant information.

2020.05.18 16:48:07 WARNING org.apache.kafka.clients.NetworkClient Thread[kafka-producer-network-thread | producer-23,5,main]: [Producer clientId=producer-23] Error while fetching metadata with correlation id 62 : {unexistingTopic2=UNKNOWN_TOPIC_OR_PARTITION}

Signed-off-by: Jorge Bescos Gascon <jorge.bescos.gascon@oracle.com>
@jbescos jbescos merged commit c9cc6f0 into helidon-io:master May 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants