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-34118 fsp_alloc_free_extent() fails to flag DB_OUT_OF_FILE_SPACE #3249

Merged
merged 1 commit into from
May 10, 2024

Conversation

dr-m
Copy link
Contributor

@dr-m dr-m commented May 10, 2024

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

Description

fsp_alloc_free_extent() failed to set *err = DB_OUT_OF_FILE_SPACE when the tablespace is full.

Release Notes

InnoDB could fail to report ER_RECORD_FILE_FULL when running out of space.

How can this PR be tested?

./mtr innodb.temporary_table,4k

Basing the PR against the correct MariaDB version

  • This is a new feature and the PR is based against the latest MariaDB development branch.
  • This is a bug fix and the PR is based against the earliest maintained branch in which the bug can be reproduced.

This is a bug fix that depends on earlier refactoring in MariaDB Server 10.6.

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.

fsp_alloc_free_extent(): When applicable, set *err = DB_OUT_OF_FILE_SPACE.
@dr-m dr-m requested a review from Thirunarayanan May 10, 2024 09:54
@dr-m dr-m self-assigned this May 10, 2024
@dr-m dr-m merged commit 6bf2b64 into 10.6 May 10, 2024
12 of 18 checks passed
@dr-m dr-m deleted the 10.6-MDEV-34118 branch May 10, 2024 10:18
@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.

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