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

SQL: Handle nulls for field IN(v1, v2, ..) expressions #34582

Closed
matriv opened this issue Oct 18, 2018 · 1 comment
Closed

SQL: Handle nulls for field IN(v1, v2, ..) expressions #34582

matriv opened this issue Oct 18, 2018 · 1 comment
Assignees
@matriv
Labels
:Analytics/SQL SQL querying >enhancement v6.5.0 v7.0.0-beta1

Comments

@matriv
Copy link
Contributor

matriv commented Oct 18, 2018

Make sure that null values in the value list and for the field are handled correctly.

Follow up to #32955.
Depends on #34573.
@matriv matriv mentioned this issue Oct 18, 2018
Merged
@matriv matriv added the :Analytics/SQL SQL querying label Oct 18, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@matriv matriv added the v7.0.0 label Oct 18, 2018
@matriv matriv self-assigned this Oct 22, 2018
matriv added a commit to matriv/elasticsearch that referenced this issue Oct 23, 2018
@matriv
SQL: Implement null handling for IN(v1, v2, ...)
6dfd9aa 
Implemented null handling for both the value tested but also for
values inside the list of values tested against.

The null handling is implemented for local processors, painless scripts
and Lucene Terms queries making it available for `IN` expressions occuring
in `SELECT`, `WHERE` and `HAVING` clauses.

Closes: elastic#34582
@matriv matriv mentioned this issue Oct 23, 2018
Merged
matriv pushed a commit that referenced this issue Oct 24, 2018
SQL: Implement null handling for IN(v1, v2, ...) (#34750)
4c73854 
Implemented null handling for both the value tested but also for
values inside the list of values tested against.

The null handling is implemented for local processors, painless scripts
and Lucene Terms queries making it available for `IN` expressions occuring
in `SELECT`, `WHERE` and `HAVING` clauses.

Closes: #34582
matriv pushed a commit that referenced this issue Oct 24, 2018
@matriv
SQL: Implement null handling for IN(v1, v2, ...) (#34750)
54f3b69 
Implemented null handling for both the value tested but also for
values inside the list of values tested against.

The null handling is implemented for local processors, painless scripts
and Lucene Terms queries making it available for `IN` expressions occuring
in `SELECT`, `WHERE` and `HAVING` clauses.

Closes: #34582
kcm pushed a commit that referenced this issue Oct 30, 2018
@kcm
SQL: Implement null handling for IN(v1, v2, ...) (#34750)
e56859b 
Implemented null handling for both the value tested but also for
values inside the list of values tested against.

The null handling is implemented for local processors, painless scripts
and Lucene Terms queries making it available for `IN` expressions occuring
in `SELECT`, `WHERE` and `HAVING` clauses.

Closes: #34582
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matriv matriv

Labels
:Analytics/SQL SQL querying >enhancement v6.5.0 v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@colings86 @matriv @elasticmachine