-
Notifications
You must be signed in to change notification settings - Fork 175
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
Bug 1328659 - Add support for utf8=utf8mb4 (switches to dynamic/compressed row format, and changes charset to utf8mb4) ❤️ #282
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Minor changes. Otherwise conversion locally went well and am able to enter emojis into text fields.
Bugzilla/DB/Mysql.pm
Outdated
= $global{innodb_file_format} eq 'Barracuda' | ||
&& $global{innodb_file_per_table} eq 'ON' | ||
&& $global{innodb_large_prefix} eq 'ON'; | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 space or more indent here please.
Bugzilla/Config/General.pm
Outdated
default => '0', | ||
type => 't', | ||
choices => [ '1', 'utf8', 'utf8mb4' ], | ||
default => 'utf8', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be instead:
type => 's',
choices => [ 'utf8', 'utf8mb4' ],
default => 'utf8',
AND TABLE_COLLATION NOT LIKE 'utf8%' | ||
LIMIT 1", undef, $db_name); | ||
AND TABLE_COLLATION != ? | ||
LIMIT 1", undef, $db_name, $collate); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dklawren this is one of those changes I talked about. Before it was a prefix match against 'utf8', so if it was utf8mb4_unicode_blah, it would still match. In prod it appears it is only utf8_general_ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
r=dkl
Checked out the latest changes and after restart of DB everything checked out. r=dkl |
No description provided.