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

Wrong $timeFilter expansion #25

Closed
Perlovka opened this issue Nov 19, 2017 · 4 comments
Closed

Wrong $timeFilter expansion #25

Perlovka opened this issue Nov 19, 2017 · 4 comments
Assignees
Labels

Comments

@Perlovka
Copy link

When grafana time selector is set for example to 'Yesterday' (2017-11-18 00:00:00 to 2017-11-18 23:59:59), 'WHERE $timeFilter' is expanded to:

WHERE event_date BETWEEN toDate(1510963200) AND toDate(1511049600) AND event_time BETWEEN toDateTime(1510963200) AND toDateTime(1511049600)

so event_date will be from 18-11-2017 to 19-11-2017 and event_time will be from 18-11-2017 00:00:00 to 19-11-2017 00:00:00 inclusive.

The right part of the time expression should be either exclusive or value of timestamp decreased by 1 second.

@hagen1778 hagen1778 self-assigned this Nov 20, 2017
@hagen1778
Copy link
Collaborator

@hagen1778 hagen1778 added the bug label Nov 20, 2017
@Perlovka
Copy link
Author

Perlovka commented Nov 20, 2017

Hi,

In 1.2.2 some files are missing:

partials/query.editor.html
partials/query.options.html

Can't edit queries ;)

And time selection problem still persist. When time range is set as full period (e.g. now-10m/m - now-10m/m), time selector tooltip shows (2017-11-20 11:35:00 - 2017-11-20 11:35:59) but expanded time query is (2017-11-20 11:35:00 - 2017-11-20 11:36:00) inclusive

http://docs.grafana.org/reference/timerange/#time-units

@hagen1778
Copy link
Collaborator

hagen1778 commented Nov 20, 2017

Indeed, I forgot to commit copied to src query.editor.html
query.options.html is dropped

And time selection problem still persist

I believe that it's just a bad release and cached files. Pls, try https://github.com/Vertamedia/clickhouse-grafana/releases/tag/1.2.3

@Perlovka
Copy link
Author

Perfect! Thank you )

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

No branches or pull requests

2 participants