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

Group by no longer supports now() as offset_interval #9142

Closed
jurom opened this issue Nov 21, 2017 · 6 comments
Closed

Group by no longer supports now() as offset_interval #9142

jurom opened this issue Nov 21, 2017 · 6 comments

Comments

@jurom
Copy link

jurom commented Nov 21, 2017

Bug report

This used to work in version 1.3.7, and I'm not sure if it was removed deliberately, or by accident (it was not mentioned in the [CHANGELOG] )(https://github.com/influxdata/influxdb/blob/master/CHANGELOG.md)

System info: InfluxDB version 1.4.2,

Steps to reproduce: Any query with offset in grouping by time set to now():

SELECT sum(amount) FROM payments WHERE time > now() - 2w AND time < now() GROUP BY time(1d, now())

Expected behavior: Results grouped by days, offset to current time.

Actual behavior: ERR: invalid time dimension offset: now()

@tux-00
Copy link

tux-00 commented Dec 12, 2017

Same here.

@aidnguyen
Copy link

+1

@jsternberg
Copy link
Contributor

This was not intentional. I'm sorry this took so long for me to see. I'll look into this so we can restore this functionality.

@tux-00
Copy link

tux-00 commented Mar 12, 2018

@jsternberg Any news on this issue ?

@vmironenko
Copy link

+1

@jsternberg
Copy link
Contributor

This is being backported for 1.5.2. We will unfortunately not be backporting this to the 1.4.x series.

Backport PR is #9651.

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

No branches or pull requests

5 participants