Skip to content

Fixed mysqldump warning for unknown option on MariaDB 11.4/11.8#238

Merged
AaronFeledy merged 2 commits intomainfrom
fix/mariadb-remove-mysqldump-config
Feb 24, 2026
Merged

Fixed mysqldump warning for unknown option on MariaDB 11.4/11.8#238
AaronFeledy merged 2 commits intomainfrom
fix/mariadb-remove-mysqldump-config

Conversation

@AaronFeledy
Copy link
Member

@AaronFeledy AaronFeledy commented Feb 24, 2026

Summary

Removes the [mysqldump] block from mariadb-compat-install.sh. The loose-skip-column-statistics option was causing warnings on MariaDB 11.4/11.8 because mariadb-dump doesn't support the loose- prefix convention.

This option is only needed for MySQL 8's mysqldump, which is already correctly handled in mysql-client-install.sh.

Background

PR #220 changed skip-column-statisticsloose-skip-column-statistics assuming MariaDB would silently ignore the loose- prefix. It doesn't — that's a MySQL-only feature. Since the wrapper scripts map mysqldumpmariadb-dump, MariaDB users hit the warning every time.

Fixes #237


Note

Low Risk
Low risk: removes a MariaDB client config block that only triggered warnings and does not change core app logic or security-sensitive behavior.

Overview
Removes the [mysqldump] configuration (including loose-skip-column-statistics) from the MariaDB compatibility install script to stop mysqldump/mariadb-dump warnings on MariaDB 11.4/11.8.

Updates the changelog to note the MariaDB warning fix (issue #237).

Written by Cursor Bugbot for commit bcb296f. This will update automatically on new commits. Configure here.

The [mysqldump] block with loose-skip-column-statistics was causing
warnings on MariaDB 11.4/11.8 because mariadb-dump doesn't support
the loose- prefix. This option is only needed for MySQL 8's mysqldump,
which is already handled in mysql-client-install.sh.

Fixes #237
@netlify
Copy link

netlify bot commented Feb 24, 2026

Deploy Preview for lando-php ready!

Name Link
🔨 Latest commit bcb296f
🔍 Latest deploy log https://app.netlify.com/projects/lando-php/deploys/699da4c1d378540008572c28
😎 Deploy Preview https://deploy-preview-238--lando-php.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 87 (🔴 down 5 from production)
Accessibility: 98 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@AaronFeledy AaronFeledy changed the title fix: remove mysqldump config from MariaDB compat script Fixed mysqldump warning for unknown option on MariaDB 11.4/11.8 Feb 24, 2026
@AaronFeledy AaronFeledy merged commit 0f73577 into main Feb 24, 2026
50 checks passed
@AaronFeledy AaronFeledy deleted the fix/mariadb-remove-mysqldump-config branch February 24, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Running mysqldump on mariadb (11.4/11.8) gives Warning for unknown option '--loose-skip-column-statistics'

1 participant