Skip to content
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

Use IN instead of EXISTS to resolve to-n relation filters #110

Merged
merged 1 commit into from
Jul 21, 2023
Merged

Use IN instead of EXISTS to resolve to-n relation filters #110

merged 1 commit into from
Jul 21, 2023

Conversation

nilmerg
Copy link
Member

@nilmerg nilmerg commented Jul 6, 2023 •
edited
Loading

@nilmerg nilmerg self-assigned this Jul 6, 2023
@cla-bot cla-bot bot added the cla/signed label Jul 6, 2023
@nilmerg nilmerg linked an issue Jul 7, 2023 that may be closed by this pull request
IcingaDB - poor performance when using custom variables in the queries Icinga/icingadb-web#711
Closed
@nilmerg nilmerg marked this pull request as ready for review July 10, 2023 08:41
@nilmerg
Copy link
Member Author

nilmerg commented Jul 10, 2023

Tested with Mysql (8 and 5.7) as well as Postgres. With and without compound keys. Works fine.

@nilmerg nilmerg added the bug Something isn't working label Jul 10, 2023
@nilmerg nilmerg added this to the v0.6.0 milestone Jul 10, 2023
@nilmerg nilmerg mentioned this pull request Jul 11, 2023
Merged
@nilmerg nilmerg merged commit 67c12ad into main Jul 21, 2023
@nilmerg nilmerg deleted the exists-to-in branch July 21, 2023 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nilmerg nilmerg

Labels
bug Something isn't working cla/signed
Projects
None yet
Milestone
v0.6.0
Development

Successfully merging this pull request may close these issues.

IcingaDB - poor performance when using custom variables in the queries
1 participant
@nilmerg