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
Fix issue #22240: Running cron jobs fails if database name is too long #25472
Fix issue #22240: Running cron jobs fails if database name is too long #25472
Conversation
Create short name for Database Lock name
Hi @vpashovski. Thank you for your contribution
For more details, please, review the Magento Contributor Guide documentation. |
Fix Test to lock database with long name
🙏 |
Hi @vpashovski I've checked PR changes and looks like all works correctly. Did I misunderstand something? @vpashovski @pmclain Could you take a look? Thanks! |
Hi @engcom-Alfa I think you are right - the check is not necessary anymore. I will remove it and commit. |
Looks good, go go go merge! :) |
Hi @pmclain, thank you for the review. |
Hi @vpashovski, thank you for your contribution! |
Description (*)
Create short name for Database Lock name.
Use same logic to generate name as Foreign keys if name is longer than 64 characters.
Fixed Issues (if relevant)
Contribution checklist (*)