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

[1.2 regression] Math on multiple selectors returns distinct points #8167

Closed
kostasb opened this issue Mar 20, 2017 · 4 comments
Closed

[1.2 regression] Math on multiple selectors returns distinct points #8167

kostasb opened this issue Mar 20, 2017 · 4 comments
Assignees
Labels
area/influxql Issues related to InfluxQL query language regression support
Milestone

Comments

@kostasb
Copy link

kostasb commented Mar 20, 2017

v1.2.1 behavior:

> insert metrics value=1
> insert metrics value=2
> SELECT min(value),max(value) from metrics
name: metrics
time min max
---- --- ---
0    1   2

> SELECT min(value)*1,max(value)*1 from metrics
name: metrics
time                min max
----                --- ---
1490013334977108401 1   
1490013337400734540     2

1.1.4 behavior:

> insert metrics value=1
> insert metrics value=2
> SELECT min(value),max(value) from metrics
name: metrics
time	min	max
----	---	---
0	1	2

> SELECT min(value)*1,max(value)*1 from metrics
name: metrics
time	min	max
----	---	---
0	1	2

May be related to #8072

@jsternberg

@kostasb kostasb added area/influxql Issues related to InfluxQL query language support regression labels Mar 20, 2017
@DAfanasyev
Copy link

Looks like it is third issue about the same regression:
#8164

@jsternberg jsternberg self-assigned this Mar 20, 2017
@jsternberg
Copy link
Contributor

Bisect says it was introduced by subqueries. I'll be looking into this. I'll close the other issues to consolidate around this one.

Bisect says it's this commit, but it's a large one: d7c8c7c.

@jsternberg
Copy link
Contributor

If we don't backport a fix for this, you can use subqueries as a workaround.

> select max*1, min*1 from (select max(value), min(value) from metrics)
name: metrics
time max min
---- --- ---
0    2   1

I seem to have isolated the issue. @jwilder should we backport this fix?

@timhallinflux
Copy link
Contributor

@rkuchan -- please add this to the release notes for 1.2.2 enterprise as a known issue -- along with the proposed workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/influxql Issues related to InfluxQL query language regression support
Projects
None yet
Development

No branches or pull requests

5 participants