perf: filter by empty array should not run a query, [skip ci] #237

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.

Sign up for GitHub

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

Jump to bottom

Draft

MrSwitch wants to merge 1 commit into main from perf-empty-values-ignore

Owner

MrSwitch commented Jun 14, 2022 •

edited

Loading

Where there are empty values, i.e.

filter: {
	'username': []
},

the query doesn't need to run, instead it does...

 WHERE
	a.username AND false


          perf: filter by empty array should not run a query, [skip ci]

b996775

MrSwitch marked this pull request as draft

June 14, 2022 12:25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet