-
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
Fix typo which trigger performance bug #35216
Fix typo which trigger performance bug #35216
Conversation
Fix bug when Category Products Count cannot be taken from catalog_category_product_index table
Hi @Dren7755. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Run only required test builds during development. Run all test builds before sending your pull request for review. For more details, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
4 similar comments
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
Thanks for the contribution @Dren7755 and @nuzil ! Tried to reproduce the issue in Pre Conditions
Reproducible steps
Expected Issue as per PR description/ comments:
Actual Result
As per the description, if my way of execution to identify the issue is deviated, kindly instruct me the correct approach to reproduce the issue in |
@magento run Functional Tests CE, Integration Tests, Static Tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run Functional Tests EE |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
It took 8 months to merge 🤣 Finally, thank you, guys! |
I just applied this patch, and run before/after tests, and it does not improve anything. I still have the same huge number of DB call to count products. (tested on global scope, and on a store scope, on a magento 2.4.5-p1 instance) It seems the See my previous comment : #35216 (comment) Am I wrong somewhere ? Can we have other feedback on this patch ? |
@VladyslavSikailo can you have a look at my comment above ? |
Hello @engcom-Hotel and @engcom-Alfa, I think this comment and latests comments here should get your attention since #35216 is now in 2.4.7-beta... I'm pretty sure it does not solve anything. Also, this is probably related to #15187 too. |
Even with this MR merged, opening Catalog > Categories from admin throws the below error. Issue:
Statement to check. Although,
This should be as follows:
Not an issue with this MR, but there is definitely an issue with some of the getTableName() Magento methods from some of the class. |
Even with this change, the original problem still persists : |
Description
Fix typo which triggers bug when Category Products Count cannot be taken from catalog_category_product_index table
Manual testing scenarios
Contribution checklist (*)
Resolved issues: