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

c/topics_frontend: break the loop when dispatching to current leader #17544

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

mmaslankaprv
Copy link
Member

@mmaslankaprv mmaslankaprv commented Apr 2, 2024

Changed handling of finish_partition_update_request to break the dispatch loop when one node leader information is stale. Now when node receives the finish_partition_update_request via RPC service it will not redirect if further to the current controller preventing the request to be infinitely redirected when node leader information is stale.

Backports Required

  • none - not a bug fix
  • none - this is a backport
  • none - issue does not exist in previous branches
  • none - papercut/not impactful enough to backport
  • v23.3.x
  • v23.2.x

Release Notes

  • none

bharathv
bharathv previously approved these changes Apr 2, 2024
ztlpn
ztlpn previously approved these changes Apr 2, 2024
src/v/cluster/topics_frontend.h Outdated Show resolved Hide resolved
Changed handling of `finish_partition_update_request` to break the
dispatch loop when one node leader information is stale. Now when node
receives the `finish_partition_update_request` via RPC service it will
not redirect if further to the current controller preventing the request
to be infinitely redirected when node leader information is stale.

Signed-off-by: Michał Maślanka <michal@redpanda.com>
@vbotbuildovich
Copy link
Collaborator

@mmaslankaprv mmaslankaprv merged commit 90483d0 into redpanda-data:dev Apr 3, 2024
17 checks passed
@mmaslankaprv mmaslankaprv deleted the break-redirection branch April 3, 2024 12:48
@vbotbuildovich
Copy link
Collaborator

/backport v23.3.x

@vbotbuildovich
Copy link
Collaborator

/backport v23.2.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants