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.

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

[backport] [22.3.x]: rm_stm/seq_table: unlink before destruction #10343

Merged
merged 2 commits into from
Apr 26, 2023
Merged

[backport] [22.3.x]: rm_stm/seq_table: unlink before destruction #10343

merged 2 commits into from
Apr 26, 2023

Conversation

bharathv
Copy link
Contributor

@bharathv bharathv commented Apr 25, 2023 •
edited
Loading

Backport of #8831

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.1.x
  • v22.3.x
  • v22.2.x

Release Notes

  • none

@bharathv
rm_stm: factor seq_table emptying into a method
051655e 
No logic changes.

(cherry picked from commit c8eeb2d)
@bharathv
rm_stm/seq_table: unlink before destruction
1c2fb4c 
Forces an unlink of each seq_table entry before destroying the object.
Makes log_state non-copyable/movable to avoid unintended effects on the
instrusive list.

(cherry picked from commit d5e4d2b)
@vshtokman vshtokman added this to the v22.3.17 milestone Apr 25, 2023
@bharathv
Copy link
Contributor Author

Failures

@mmaslankaprv mmaslankaprv merged commit be3d3fd into redpanda-data:v22.3.x Apr 26, 2023
@BenPope BenPope added the kind/backport PRs targeting a stable branch label Apr 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mmaslankaprv mmaslankaprv mmaslankaprv approved these changes

Assignees
No one assigned
Labels
area/redpanda kind/backport PRs targeting a stable branch
Projects
None yet
Milestone
v22.3.17
Development

Successfully merging this pull request may close these issues.
4 participants
@bharathv @mmaslankaprv @BenPope @vshtokman