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

Skip refresh metadata for errored topic, fix #806 #820

Merged
merged 2 commits into from
Jan 13, 2022

Conversation

1000Delta
Copy link
Contributor

@1000Delta 1000Delta commented Jan 7, 2022

Skip refresh metadata for errored topic, fixed the problem metadata refresh indefinitely,
such as write message to a topic that not exist, but kafka disabled topic auto creation.

Fixes #806

Refer to the implementation of #673

Skip refrash metadata in roundTrip about errored topic,
such as write message to a not exist topic,
kafka will response error UNKNOWN_TOPIC_OR_PARTITION to the  topic,
it causes indefinitely metadata refreshing.

Fixes segmentio#806
Copy link
Contributor

@achille-roussel achille-roussel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for investigating this issue and submitting a fix!

The changes look good and well tested 👍

@achille-roussel achille-roussel merged commit f487e01 into segmentio:main Jan 13, 2022
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.

WriteMessages function will block the goroutine while topic not exist and client no permission to create topic
2 participants