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

[Feat Req] Add the possibility to include times extremity #8319

Closed
Oxydros opened this Issue May 8, 2017 · 0 comments

Comments

Projects
None yet
2 participants
@Oxydros
Contributor

Oxydros commented May 8, 2017

  • Grafana 4.2.0
  • InfluxDB

I think the data queried in InfluxDB by Grafana exclude the extremities.
For example, if I want to query between 2017-04-01 00:00:00 and 2017-04-30 23:59:59, the query use the operand '>' and '<' in the where clause. I lost the data at 2017-04-01 00:00:00 and the data at 2017-04-30 23:59:59, witch give me bad results.

I think the code responsible for this is here:

func (query *Query) renderTimeFilter(queryContext *tsdb.QueryContext) string {

Do you think that it's possible to add a checkbox in the dashboard configuration to choose if the user want to include or exclude the time extremity ?

I started to work on that on my local fork.
At first I just want to print the string returned by fmt.Sprintf using fmt.Printf, but I can't figure out how to do this.
I can print thinks when I put them in the init functions, like in response_parser.go for example, but not in the Build function of query.go

Do you have an Idea why ?

I use bra and grunt, as suggested in the readme

@torkelo torkelo changed the title from [FEATURE REQUEST] Add the possibility to include times extremity to [Feat Req] Add the possibility to include times extremity May 9, 2017

@torkelo torkelo closed this Aug 7, 2017

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