-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Fix issue Redis doesn't store GraphQl schema cache #39172
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
base: 2.4-develop
Are you sure you want to change the base?
Conversation
This issue happen because GraphQl schema cache tags include item ['EAV' => 'EAV']
Hi @m08110071. Thank you for your contribution! Add the comment under your pull request to deploy test or vanilla Magento instance:
❗ Automated tests can be triggered manually with an appropriate comment:
Allowed build names are:
You can find more information about the builds here For more details, review the Code Contributions documentation. |
@magento run all tests |
@magento run Unit Tests |
@m08110071 Please sign CLA |
@magento run Integration Tests |
@m08110071 we have the same issue after upgrade Magento from 2.4.6-p8 to 2.4.7-p3 - all GraphQl queries take more than 7 seconds to execute because of problem with save data to Redis - there is example from New Relic: After applying your patch from here #39172, loading time dropped to about 100ms! I think this does not happen on every instance, you need to set the L2 cache and permanent cache settings (using an external Redis server and local per app instance): https://experienceleague.adobe.com/en/docs/commerce-operations/configuration-guide/cache/level-two-cache |
@m08110071 Could you please sign CLA, because we can't proceed with this PR without it. |
I tested changes locally, all look good for me |
This issue happen because GraphQl schema cache tags include item ['EAV' => 'EAV']
Description (*)
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)