-
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
Removed redundant Gallery subscription for catalog rules #22320
Removed redundant Gallery subscription for catalog rules #22320
Conversation
Hi @VitaliyBoyko. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @orlangur, thank you for the review. |
Hi @dmytro-ch, thank you for the review. |
✔️ QA passed |
Hi @VitaliyBoyko, thank you for your contribution! |
Description (*)
Hi @magento-engcom-team
I suggest removing 'catalog_product_entity_gallery' subscription table which is not related to price/price-rules from CatalogRule mview.
Listening 'catalog_product_entity_gallery' causing a huge amount of records to changelog when saving the product with applied price rule in case the product has many product images.
Gallery attribute does not contain data related to pricing framework and can't be utilized creating price rule condition, so, I don't see a reason to listen to that table for the partial reindex changelog.
Thank you!
Manual testing scenarios (*)
N/A
Contribution checklist (*)