Skip to content

MDEV-35525: Index corruption in reverse scans - #3669

Merged
dr-m merged 1 commit into
10.6from
10.6-MDEV-35525
Dec 3, 2024
Merged

MDEV-35525: Index corruption in reverse scans#3669
dr-m merged 1 commit into
10.6from
10.6-MDEV-35525

Conversation

@dr-m

@dr-m dr-m commented Nov 29, 2024

Copy link
Copy Markdown
Contributor
  • The Jira issue number for this PR is: MDEV-35525

Description

btr_cur_t::search_leaf(): In the BTR_SEARCH_PREV and BTR_MODIFY_PREV modes, reset the previous search status before invoking page_cur_search_with_match(). Otherwise, we the search could invoke in a totally wrong subtree.

Release Notes

See MDEV-35525.

How can this PR be tested?

The bug was reproduced as part of testing #3562 in the random query generator (RQG) by using innodb_change_buffering=all and innodb_page_size=4k. Possibly also using innodb_change_buffering_debug.

Basing the PR against the correct MariaDB version

  • This is a new feature or a refactoring, and the PR is based against the main branch.
  • This is a bug fix, and the PR is based against the earliest maintained branch in which the bug can be reproduced.

PR quality check

  • I checked the CODING_STANDARDS.md file and my PR conforms to this where appropriate.
  • For any trivial modifications to the PR, I am ok with the reviewer making the changes themselves.

btr_cur_t::search_leaf(): In the BTR_SEARCH_PREV and BTR_MODIFY_PREV
modes, reset the previous search status before invoking
page_cur_search_with_match(). Otherwise, we the search could invoke
in a totally wrong subtree.

This fixes a regression that was introduced in
commit de4030e (MDEV-30400).
@dr-m
dr-m requested a review from vlad-lesin November 29, 2024 13:18
@dr-m dr-m self-assigned this Nov 29, 2024
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@dr-m
dr-m merged commit 1a9011d into 10.6 Dec 3, 2024
@dr-m
dr-m deleted the 10.6-MDEV-35525 branch December 3, 2024 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants