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.

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

SQL: improvements to negative numbers #33765

Closed
costin opened this issue Sep 17, 2018 · 1 comment

Comments

@costin
Copy link
Member

commented Sep 17, 2018

Currently negative numbers are handled as Not(number) which complicates the plans considerably and introduces bugs in folding (such as ordering).

@elasticmachine

This comment has been minimized.

Copy link
Collaborator

commented Sep 17, 2018

Pinging @elastic/es-search-aggs

costin added a commit to costin/elasticsearch that referenced this issue Sep 17, 2018
SQL: Grammar tweak for number declarations
Consider plus and minus as part of a number declaration (to avoid the
minus be treated as a negation).

Close elastic#33765
costin added a commit that referenced this issue Sep 18, 2018
SQL: Grammar tweak for number declarations (#33767)
Consider plus and minus as part of a number declaration (to avoid the
minus be treated as a negation).

Close #33765
costin added a commit that referenced this issue Sep 18, 2018
SQL: Grammar tweak for number declarations (#33767)
Consider plus and minus as part of a number declaration (to avoid the
minus be treated as a negation).

Close #33765

(cherry picked from commit ab9c28a)

@colings86 colings86 added v7.0.0-beta1 and removed v7.0.0 labels Feb 7, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.