-
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
[Issue] Fix usage of EAV cache type #32322
Comments
Hi @engcom-Hotel. Thank you for working on this issue.
|
Hello @ilnytskyi, Thank you for your report. To speed up the processing of this issue, please provide us with the following information:
Thanks |
Dear @ilnytskyi, We have noticed that this issue has not been updated for a period of 14 Days. Hence we assume that this issue is fixed now, so we are closing it. Please raise a fresh ticket or reopen this ticket if you need more assistance on this. Regards |
✅ Jira issue https://jira.corp.adobe.com/browse/AC-13355 is successfully created for this GitHub issue. |
✅ Confirmed by @engcom-Dash. Thank you for verifying the issue. |
Unfortunately, not enough information was provided to create a Jira ticket. Please make sure you added the following label(s): Once all required labels are present, please add |
❌ Cannot export the issue. This GitHub issue is already linked to Jira issue(s): https://jira.corp.adobe.com/browse/AC-13355 |
This issue is automatically created based on existing pull request: #31264: Fix usage of EAV cache type
Description (*)
This PR fixes usage of EAV cache type in some places
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
configured_eav_cache
virtual typeUPD: looks like it's bigger problem I see many other classes do not use
\Magento\Eav\Model\Cache\Type
and they can't be fixed due toMagento\Framework\App\CacheInterface
andMagento\Framework\Cache\FrontendInterface
incompatibilities.It should be figured out and fixed in different PR/issue.
\Magento\Catalog\Plugin\Model\ResourceModel\Config
checks if EAV cache enabled but store data in default cache type. This PR fixes missed configurationQuestions or comments
Contribution checklist (*)
The text was updated successfully, but these errors were encountered: