Skip to content
/ server Public

MDEV-38327 Do not use rowid filter when switching to index merge#4541

Merged
spetrunia merged 2 commits into11.4from
bb-11.4-mdev-38327
Jan 26, 2026
Merged

MDEV-38327 Do not use rowid filter when switching to index merge#4541
spetrunia merged 2 commits into11.4from
bb-11.4-mdev-38327

Conversation

@mariadb-YuchenPei
Copy link
Contributor

Index merge and rowid filter should not be used together, however, even if index merge is not chosen earlier in best_access_path, it may be chosen again in make_join_select. Therefore this patch ensures that rowid filter is not used when index merge is chosen in make_join_select.

@CLAassistant
Copy link

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.

@mariadb-YuchenPei mariadb-YuchenPei force-pushed the bb-11.4-mdev-38327 branch 3 times, most recently from d3143e5 to 47c2dfc Compare January 21, 2026 00:49
Copy link
Member

@spetrunia spetrunia left a comment

Choose a reason for hiding this comment

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

Looks good now.

factor out common index merge checks of quick select types
…hod is index merge

Index merge and rowid filter should not be used together, however,
even if index merge is not chosen earlier in best_access_path, it may
be chosen again in make_join_select, inside ref_to_range. Therefore
this patch ensures that rowid filter is not used when index merge is
chosen there.
@spetrunia spetrunia merged commit 2c2a418 into 11.4 Jan 26, 2026
15 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants