Closed
Description
This issue is automatically created based on existing pull request: #29376: Checkout / Sales Rules / Get rid of redundant DB query
Preconditions:
- Loading data in the loop?
- No, thank you!
By replacing direct SQL query with Data Provider that was already used in this Validator class allowed me to drastically reduce the cost of checkout placing process.
Local environment
20 products in cart
Custom project
22 products in cart
Related Pull Requests
Fixed Issues (if relevant)
Performance of Checkout
Steps to reproduce:
- Install sample data
- Create Catalog rule based on category
- Add plenty of products (15 - inf.)
- Place an order.
Expected Result:
Checkout should work faster by 20 - 30%
Questions or comments
Contribution checklist (*)
- Pull request has a meaningful description of its purpose
- All commits are accompanied by meaningful commit messages
- All new or changed code is covered with unit/integration tests (if applicable)
- All automated tests passed successfully (all builds are green)
Metadata
Metadata
Assignees
Labels
The issue has been fixed in 2.4-develop branchGate 3 Passed. Manual verification of the issue completed. Issue is confirmedGate 1 Passed. Automatic verification of issue format passedGate 4. Acknowledged. Issue is added to backlog and ready for developmentOnce P0 defects have been fixed, a defect having this priority is the next candidate for fixing.The issue has been reproduced on latest 2.4-develop branchMajor restrictions or short-term circumventions are required until a fix is available.Issue related to Developer Experience and needs help with Triage to Confirm or Reject it