-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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 layout xml and page layout caching issue on redis cluster under high load #22765
Conversation
(build 2.3.0.065)
(build 2.3.1.104)
…igh load Bugs which were fixed: - $this->pageLayout was not checked after reading from cache, but was used as is - two cache items were used in once place instead of one (performance impact) Changes: - replace 2 cache items by 1 - it should improve performance - add "_MERGED" to cache key suffix to have compatibility with old cache keys during deployment of new version
…ework\View\Model\Layout\Merge
Hi @andrey-legayev. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
|
I've created a branch from tag 2.3.1 (which is branch "2.2") and pointed pull request to 2.3-develop |
I'm not sure how this works I need help |
I've cherry-picked all my commits to be on top of 2.3-develop and created #22766 |
Hi @andrey-legayev, thank you for your contribution! |
Description (*)
Bugs which were fixed:
Changes:
Fixed Issues (if relevant)
Manual testing scenarios (*)
N/A
Contribution checklist (*)