-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[0.10.0] panic in model.Points.scanTo #5664
Comments
index out of range
panic
@jwilder fixed this. Though it's slightly perplexing because |
Alas, I don't have the full log leading up to the crash. All I have is this snippet just before the backtrace in the gist:
I hope this helps. |
@e-dard just in case it is useful to you, I have a query that caused this in my database. select count(value)/60 into "default"."count.foobar" from "foobar.state" where time > now() - 24h group by time(5m), foo, bar The series "foobar.state" has to exist and have data in the range specified by the time filter - it won't crash otherwise. |
Previously, influx would either panic (influxdata#5664) or enter a loop (post-influxdata#5697) in the case that a binary point contained an empty field name. Now we tolerate the case by skipping the empty field. Signed-off-by: Jon Seymour <jon@wildducktheories.com>
Note that this series also includes cherry-pick of influxdata#5697. Signed-off-by: Jon Seymour <jon@wildducktheories.com>
I'm using
telegraf
feeding intoinfluxdb
. In order to compute the% CPU
usage of a container as a time series, it is necessary to compute the change in measurements likedocker_cpu.total_usage
. Since expressions of aggregates and the functiondifference()
aren't supported yet, I thought of using a continuous query to precompute these deltas. Basically, within aGROUP BY
time interval, computeLAST(total_usage) - FIRST(total_usage)
(orMAX(total_usage) - MIN(total_usage)
, they should be the same).What I'm running into is a panic and
influxdb
crashes.influxdb
?I've included the backtrace and the query in this gist.
The text was updated successfully, but these errors were encountered: