Skip to content

KAFKA-19455: Retry persister request on metadata image issues. #20078

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Jul 1, 2025
Merged

KAFKA-19455: Retry persister request on metadata image issues. #20078

merged 1 commit into from
Jul 1, 2025
+7 −0

Conversation

smjn
Copy link
Collaborator

@smjn smjn commented Jul 1, 2025 •
edited by AndrewJSchofield
Loading

  • If we get an UNKNOWN_TOPIC_OR_PARTITION response from the
    ShareCoordinator is could imply a transient issue where the metadata
    image is not upto date.
  • In this case it makes sense to retry the request to give time for data
    to be available.
  • In this PR, we are making the required change.

Reviewers: Andrew Schofield aschofield@confluent.io
@github-actions github-actions bot added the triage PRs from the community label Jul 1, 2025
@smjn smjn requested a review from AndrewJSchofield July 1, 2025 09:33
@github-actions github-actions bot added core Kafka Broker KIP-932 Queues for Kafka small Small PRs labels Jul 1, 2025
@smjn smjn added ci-approved and removed core Kafka Broker triage PRs from the community labels Jul 1, 2025
AndrewJSchofield
AndrewJSchofield approved these changes Jul 1, 2025
View reviewed changes
Copy link
Member

@AndrewJSchofield AndrewJSchofield 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 the PR.
@AndrewJSchofield AndrewJSchofield merged commit ac583ad into apache:trunk Jul 1, 2025
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci-approved KIP-932 Queues for Kafka small Small PRs
2 participants
@smjn @AndrewJSchofield