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

MDEV-34919 - Aria crashes with high-level (vector) indexes #3570

Open
wants to merge 1 commit into
base: bb-11.6-MDEV-32887-vector
Choose a base branch
from

Conversation

svoj
Copy link
Contributor

@svoj svoj commented Oct 10, 2024

  • The Jira issue number for this PR is: MDEV-34919

Description

Fixes aria crash with high-level indexes.

Release Notes

None.

How can this PR be tested?

mtr vector

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.

@svoj svoj requested a review from vuvova October 10, 2024 14:39
@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.

@svoj svoj force-pushed the bb-11.6-MDEV-34919-vector-aria branch from 777727b to 3a5711c Compare October 14, 2024 06:55
@svoj svoj force-pushed the bb-11.6-MDEV-34919-vector-aria branch from 3a5711c to 7538ff5 Compare October 15, 2024 12:04
@vuvova vuvova force-pushed the bb-11.6-MDEV-32887-vector branch 4 times, most recently from 4b57276 to 4dfa605 Compare October 19, 2024 12:47
Since high-level index tables do not participate in thr_multi_lock(), added
explicit call to THR_LOCK::start_trans(). This is needed mostly for Aria to
handle transaction logging.
@svoj svoj force-pushed the bb-11.6-MDEV-34919-vector-aria branch from 7538ff5 to 670f3a6 Compare October 20, 2024 20:32
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.

2 participants