elastic / elastic/elasticsearch

QL: Scripts should check all docvalues not just the first

Open
#55,341 3 comments 0 reactions 0 assignees View on GitHub
:Analytics/EQL :Analytics/SQL >tech-debt Team:Analytics
Dominant language
Java
Stars
77.9k
Forks
26.1k
PR merge metrics
PR metrics pending

Description

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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.