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

Jump to bottom

innodb_file_format is checked case-sensitive #16985

Closed
greno2 opened this issue Jun 29, 2021 · 2 comments
Closed

innodb_file_format is checked case-sensitive #16985

greno2 opened this issue Jun 29, 2021 · 2 comments
Assignees
@williamdes
Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete
Projects
issues
Milestone
5.1.2

Comments

@greno2
Copy link

greno2 commented Jun 29, 2021

Describe the bug

By accident we set innodb_file_format in my.cnf to "barracuda" (lowercase b). PMA doesn't detect this setting correct as libraries/classes/Operations.php checks for $innodb_file_format === 'Barracuda' with uppercase B.

To Reproduce

  • Set innodb_file_format="barracuda"
  • Restart MySQL / MariaDB
  • Go to PMA and see e.g. missing rowformat=dynamic that requires Barracuda

Expected behavior

Ignore case
@williamdes williamdes added the Bug A problem or regression with an existing feature label Jun 29, 2021
@williamdes williamdes added this to the 5.1.2 milestone Jun 29, 2021
@williamdes williamdes added this to Needs triage in issues via automation Jun 29, 2021
@williamdes williamdes moved this from Needs triage to to be fixed soon in issues Jun 29, 2021
@williamdes
Copy link
Member

Thank you for the report @greno2 !

@ibennetch
Copy link
Member

👍

@iifawzi iifawzi mentioned this issue Jul 14, 2021
Merged
@williamdes williamdes self-assigned this Jul 15, 2021
@williamdes williamdes added the has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete label Jul 15, 2021
williamdes added a commit that referenced this issue Jul 15, 2021
@williamdes
Add ChangeLog entries for c81bb0a, #16976 and #16985
1692494 
Signed-off-by: William Desportes <williamdes@wdes.fr>
issues automation moved this from to be fixed soon to Closed Jul 15, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@williamdes williamdes

Labels
Bug A problem or regression with an existing feature has-pr An issue that has a pull request pending that may fix this issue. The pull request may be incomplete
Projects
issues
  
Closed
Milestone
5.1.2
Development

No branches or pull requests
3 participants
@ibennetch @greno2 @williamdes