Skip to content

OAK-9899 - Optimize in(x, y, z) constraints to use the terms ES query instead of a union of range queries#689

Merged
fabriziofortino merged 1 commit intoapache:trunkfrom
nfsantos:OAK-9899
Sep 7, 2022
Merged

OAK-9899 - Optimize in(x, y, z) constraints to use the terms ES query instead of a union of range queries#689
fabriziofortino merged 1 commit intoapache:trunkfrom
nfsantos:OAK-9899

Conversation

@nfsantos
Copy link
Contributor

@nfsantos nfsantos commented Sep 5, 2022

Change the way that the in() property restriction in SQL2 is mapped to an Elastic query: instead of using a union of range queries, one for each value in the set, use the terms query of Elastic.

… an Elastic query. Instead of using and or of range queries, one for each value in the set, use the terms query of Elastic.
Copy link
Contributor

@nit0906 nit0906 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@fabriziofortino fabriziofortino merged commit a0bd343 into apache:trunk Sep 7, 2022
@nfsantos nfsantos deleted the OAK-9899 branch September 7, 2022 09:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants