-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
Set default collation to utf8mb4 #35174
base: 2.4-develop
Are you sure you want to change the base?
Conversation
Hi @sippsolutions. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Run only required test builds during development. Run all test builds before sending your pull request for review. For more details, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
Hi @sippsolutions |
@Den4ik did you see my changes? the tests are intended to fail. |
Hi @sippsolutions |
Hi @Den4ik, |
Hi @sippsolutions |
@Den4ik that would be awesome :) |
@sippsolutions Could you resolve merge conflict? |
This might be a bit more complex than what it sounds like. See OpenMage/magento-lts#1036 where an attempt is made to apply a similar change to Magento 1, it contains a bunch of interesting information in the comments. |
Would also need changes, otherwise you'll get mismatched collations which will kill your performance. |
Hello @sippsolutions |
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.
Functional tests passed in previous runs.
Checked DB tests and it looks ok
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.
Approved
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run Functional Tests CE, Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
Hi, Internal team has started working on this issue. Most of the code changes are complete. We are trying to get conclusion over DBC failure and are in discussion with PO. Thanks. |
As per this comment #35174 (comment), we are moving this issue We will update you here about the progress. Thanks |
Hi, All the required changes are done and working with Engineering Manager to speed up. Delivery is aligned for Oct Release. Thanks. |
Description (*)
MySQL 8 interprets utf8 as utf8mb3, which is deprecated:
https://dev.mysql.com/doc/relnotes/mysql/8.0/en/news-8-0-24.html
https://dev.mysql.com/doc/refman/8.0/en/charset-unicode-utf8mb3.html
We should therefor ensure utf8mb4 is used.
Contribution checklist (*)
Resolved issues: