-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Ensure integer values are not quoted as strings #18287
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
Ensure integer values are not quoted as strings #18287
Conversation
Hi @udovicic. Thank you for your contribution
For more details, please, review the Magento Contributor Assistant documentation |
Hi @udovicic! I really like this PR, would you mind changing the target branch to 2.3? We are trying to have everything in 2.3 before other branches. Thanks! |
Hey @udovicic were you able to review this and submit the PR to the 2.3 branch? To keep things in order, we are trying to have everything merged there before accepting the PR to other branches. Besides that, everything looks good! |
Hi @miguelbalparda , I have tested this only on 2.2, but yes, I can adjust for 2.3 as well. Do you want me to make to open a separate request for that one, as this branch was forked of 2.2 develop... |
A separated pull request will work. Million thanks!! |
@miguelbalparda Please see #18611 for |
Hi @orlangur, thank you for the review. |
Hi @udovicic. Thank you for your contribution. Please, consider to port this solution to 2.3 release line. |
Description
In cases where indexers does a filtering in the style of
WHERE entity_id IN (...)
there is a performance penalty if IDs are passed as strings, rather than integer values. This is especially visible on larger data sets. This pull request should cover some common occurrences of this.Fixed Issues (if relevant)
Manual testing scenarios
Contribution checklist