-
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
GraphQl Schema Is Not Cached With Redis #39168
Comments
Hi @m08110071. Thank you for your report.
Join Magento Community Engineering Slack and ask your questions in #github channel. |
@engcom-Bravo This is my PR to fix It: |
Hi @engcom-November. Thank you for working on this issue.
|
Hello @m08110071, Thank you for the report and collaboration! Not able to reproduce this on 2.4-develop with redis cache configured. Please take a look at the screenshot below: And also the same key can be seen in redis as well: Please let us know if we are missing anything. Thank you. |
Please try to with Adobe Commerce 2.4.7-p2. You can check my PR to resolve this issue to understand the issue. I debugged and see this issue happen because |
Hello @m08110071, Tried with 2.4.7-p2 community and commerce eddition with requirepass, but in both the cases this is not reproducible. Please let us know if there are any pre conditions required. Thank you. |
Hello @m08110071, This issue is being closed since it has not been updated in a long time. Thank you. |
Preconditions and environment
Steps to reproduce
Expected result
GraphQl schema is cached
Actual result
GraphQl schema is not cached
Additional information
File: vendor/magento/module-catalog-graph-ql/etc/di.xml
<virtualType name="Magento\Framework\GraphQl\Config\Data" type="Magento\Framework\Config\Data"> <arguments> <argument name="cacheTags" xsi:type="array"> <!-- Note: Because of DynamicAttributeReaders, this cache needs to be cleaned when attributes change--> <item name="EAV" xsi:type="string">EAV</item> </argument> </arguments> </virtualType>
Release note
No response
Triage and priority
The text was updated successfully, but these errors were encountered: