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

bug: fill(previous) and multiplication operator result in squaring coefficient #6911

Closed

apetryk2 opened this issue Jun 25, 2016 · 0 comments

Assignees

Labels

Milestone

apetryk2 commented Jun 25, 2016 •

edited by jsternberg

Loading

Bug report

influxdb 1.0.0-beta2
OS: windows 10

Steps to reproduce:

Create the following dummy data in a measurement:

measurement value=0 1420070401000000000
measurement value=15 1420070431000000000
measurement value=0 1420070521000000000

Execute the following query to see things look ok:

> select "value" from "measurement" where time > '2015-01-01T00:00:00Z'
2015-01-01T00:00:01Z    0
2015-01-01T00:00:31Z    15
2015-01-01T00:02:01Z    0

Execute the following query to see that regular group by of mean looks ok

> select mean("value") from "measurement" where time > '2015-01-01T00:00:00Z' and time < '2015-01-01T00:03:00Z' group by time(30s) fill(previous)
2015-01-01T00:00:00Z    0
2015-01-01T00:00:30Z    15
2015-01-01T00:01:00Z    15
2015-01-01T00:01:30Z    15
2015-01-01T00:02:00Z    0
2015-01-01T00:02:30Z    0

Multiply the mean by 4. We observe that on data points where the fill(previous) is being applied the coeficient for multiplication is increasing to 4^2.

> select 4*mean("value") from "measurement" where time > '2015-01-01T00:00:00Z' and time < '2015-01-01T00:03:00Z' group by time(30s) fill(previous)
2015-01-01T00:00:00Z    0
2015-01-01T00:00:30Z    60
2015-01-01T00:01:00Z    240   <-   this value should also be 60, no?
2015-01-01T00:01:30Z    240  <-    ditto.
2015-01-01T00:02:00Z    0
2015-01-01T00:02:30Z    0

The text was updated successfully, but these errors were encountered:

jsternberg self-assigned this

jsternberg added the area/queries label

jsternberg mentioned this issue

Fix fill(previous) when used with math operators #6918

Merged

jsternberg added this to the 1.0.0 milestone

jsternberg closed this as completed in #6918

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment