-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Selector (like max, min, first, etc) return a string instead of timestamp #4415
Comments
The below is predicated on this issue being triaged/confirmed Quick question - is there any idea of how tricky this issue would be to fix? Or any kind of workaround that is available now? The reason I'm asking is because I'm scheduled to give an internal demo of InfluxDB next week...and it might be somewhat awkward/embarrassing for me if max() doesn't work, since it's one of the main functions we use...haha. (We need to show peaks in the data). So just wondering if there's something I can do to get it to work before then. |
@victorhooi I don't see anything that indicates the timestamps are strings. What leads you to that conclusion? Can you give some samples that return only a few dozen points, and perhaps using |
@beckettsean Sure, I can give a shorter example. Sorry about the scrolling, there doesn't seem to be a way in Github to make it wordwrap, so I've manually word-wrapped with When I use
But when I use
The quotation marks around it lead me to think it's a string - and I'm not sure why it's not just an epoch like for Here is the complete dump for both: Request URL for
Response for
Request URL for
and the corresponding response:
|
@beckettsean I repeated the same query via the InfluxDB CLI shell, and also got incorrect results. Here is the
Here is the
Notice how for (I noticed your request for pretty=true - I'm assuming the above supplants that, but if you still need that, let me know). |
Thanks, @victorhooi, with both new examples I can actually see the string timestamps. When parsing the one line return earlier those didn't pop out. @corylanou @jwilder @DanielMorsing something wonky with timestamps and MAX in 0.9.5 nightly. |
Cool, glad the new examples help =). Let me know if there's anything you want me to test on this data-set, or any weird workarounds I'd be happy to guinea pig, haha. |
Always return time, never strings. fixes #4415
This was originally raised as grafana/grafana#2930 - however, it appears to be an issue in InfluxDB
tl;dr - max() is returning strings instead of timestamps
I'm attempting to use Grafana to graph some sparse points stored in InflxuDB.
I am running InfluxDB nightly (0.9.5-nightly-f1e0c59)
For some odd reason, it works fine with
mean()
, but not withmax()
.This is the graph with
mean()
:The request:
The response:
However, if I switch to
max()
, I get an empty graph:This is the request:
This is the response:
torkelo has pointed out (grafana/grafana#2930 (comment)) that the timestamps fro the two are different - for some weird reason, they're normal int epochs for
mean()
, butmax()
is giving some weird ISO 8601 timestamp as a string?The text was updated successfully, but these errors were encountered: