fix(query) various unary query bugs. - Githubissues

filodb / FiloDB

Distributed Prometheus time series database

Apache License 2.0

1.43k stars 225 forks source link

fix(query) various unary query bugs. #1746

Closed yu-shipit closed 6 months ago

yu-shipit commented 6 months ago

rdar://125373726

Pull Request checklist

[x] The commit(s) message(s) follows the contribution guidelines ?
[x] Tests for the changes have been added (for bug fixes / features) ?
[ ] Docs have been added / updated (for bug fixes / features) ?

Current behavior : does not support various queries with unary expressions.

1 + -1
-1 + 1 
-1 + -1
(1 + 1) + -1
-1 + (1 + 1)

New behavior :

The queries are supported.

BREAKING CHANGES

If this PR contains a breaking change, please describe the impact and migration path for existing applications. If not please remove this section.

Breaking changes may include:

Any schema changes to any Cassandra tables
The serialized format for Dataset and Column (see .toString methods)
Over the wire formats for Akka messages / case classes
Changes to the HTTP public API
Changes to query parsing / PromQL parsing

Other information: