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

QL: Scripts should check all docvalues not just the first #55341

Open
costin opened this issue Apr 16, 2020 · 3 comments
Open

QL: Scripts should check all docvalues not just the first #55341

costin opened this issue Apr 16, 2020 · 3 comments
Labels
:Analytics/EQL EQL querying :Analytics/SQL SQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)

Comments

@costin
Copy link
Member

costin commented Apr 16, 2020

Currently the generated scripts only check the first value returned from doc values regardless of the number of values.
In order to be aligned with the query behavior, the scripts should check that at least on value matches not just the first.
Since doc values serve as input, this change requires a different script generation strategy to go through all values.
@costin costin added :Analytics/SQL SQL querying :Analytics/EQL EQL querying labels Apr 16, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-ql (:Query Languages/SQL)

@costin costin mentioned this issue Apr 16, 2020
Closed
@jrodewig jrodewig mentioned this issue Apr 17, 2020
Closed
@rjernst rjernst added the Team:QL (Deprecated) Meta label for query languages team label May 4, 2020
@jrodewig jrodewig mentioned this issue May 14, 2020
Merged
@rw-access
Copy link
Contributor

Possibly solved by #74567

@wchaparro wchaparro removed the Team:QL (Deprecated) Meta label for query languages team label Jan 17, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Analytics/EQL EQL querying :Analytics/SQL SQL querying Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
6 participants
@costin @rjernst @wchaparro @elasticmachine @rw-access @elasticsearchmachine