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: WHERE with condition evaluating to false doesn't work #34613

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

SQL: WHERE with condition evaluating to false doesn't work #34613

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

Comments

@matriv
Copy link
Contributor

matriv commented Oct 18, 2018

When the condition in WHERE clause can be evaluated (literal expression) and evaluates to false, an exception is thrown:

java.sql.SQLFeatureNotSupportedException: Found 1 problem(s)
line 1:8: Unexecutable item

E.g.: select * from test_emp where 1>2
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

@matriv matriv mentioned this issue Oct 18, 2018
Merged
@matriv matriv self-assigned this Oct 24, 2018
matriv added a commit to matriv/elasticsearch that referenced this issue Oct 24, 2018
@matriv @costin
SQL: Fix queries with filter resulting in NO_MATCH
1483727 
Previously, `Mapper` was returning an incorrect plan which resulted in an
```
SQLFeatureNotSupportedException: Found 1 problem(s)
line 1:8: Unexecutable item
```

Queries with a `WHERE` and/or `HAVING` clause which results in NO_MATCH
are now handled correctly and return 0 rows.

Fixes: elastic#34613

Co-authored-by: Costin Leau <costin.leau@gmail.com>
@matriv matriv mentioned this issue Oct 24, 2018
Merged
matriv pushed a commit that referenced this issue Oct 24, 2018
@costin
SQL: Fix queries with filter resulting in NO_MATCH (#34812)
98cd7ca 
Previously, `Mapper` was returning an incorrect plan which resulted in an
```
SQLFeatureNotSupportedException: Found 1 problem(s)
line 1:8: Unexecutable item
```

Queries with a `WHERE` and/or `HAVING` clause which results in NO_MATCH
are now handled correctly and return 0 rows.

Fixes: #34613

Co-authored-by: Costin Leau <costin.leau@gmail.com>
matriv pushed a commit that referenced this issue Oct 24, 2018
@costin @matriv
SQL: Fix queries with filter resulting in NO_MATCH (#34812)
f04f19e 
Previously, `Mapper` was returning an incorrect plan which resulted in an
```
SQLFeatureNotSupportedException: Found 1 problem(s)
line 1:8: Unexecutable item
```

Queries with a `WHERE` and/or `HAVING` clause which results in NO_MATCH
are now handled correctly and return 0 rows.

Fixes: #34613

Co-authored-by: Costin Leau <costin.leau@gmail.com>
kcm pushed a commit that referenced this issue Oct 30, 2018
@costin @kcm
SQL: Fix queries with filter resulting in NO_MATCH (#34812)
a813671 
Previously, `Mapper` was returning an incorrect plan which resulted in an
```
SQLFeatureNotSupportedException: Found 1 problem(s)
line 1:8: Unexecutable item
```

Queries with a `WHERE` and/or `HAVING` clause which results in NO_MATCH
are now handled correctly and return 0 rows.

Fixes: #34613

Co-authored-by: Costin Leau <costin.leau@gmail.com>
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 >bug v6.5.0 v7.0.0-beta1
Projects
None yet
Milestone
No milestone
Development

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