github-mirrors/VictoriaMetrics
1
0
Fork 0
mirror of https://github.com/VictoriaMetrics/VictoriaMetrics.git synced 2025-03-11 15:34:56 +00:00
Code Issues Projects Releases Packages Wiki Activity

app/vmselect/promql: return NaN from histogram_quantile if at least a single bucket is broken

Browse source
This commit is contained in:
Aliaksandr Valialkin 2019-07-31 01:17:41 +03:00
parent 4aaa5c2efc
commit 8bd6f1f6df
1 changed files with 8 additions and 8 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
app/vmselect/promql/transform.go
View file

@ -324,6 +324,14 @@ func transformHistogramQuantile(tfa *transformFuncArg) ([]*timeseries, error) {
if math.IsNaN(phi) { if math.IsNaN(phi) {
return nan return nan
} }
// Verify for broken buckets with NaN or negative values.
for _, xs := range xss {
v := xs.ts.Values[i]
if math.IsNaN(v) || v < 0 {
// Broken bucket.
return nan
}
}
if phi < 0 { if phi < 0 {
return -inf return -inf
} }
@ -331,16 +339,8 @@ func transformHistogramQuantile(tfa *transformFuncArg) ([]*timeseries, error) {
return inf return inf
} }
vReq := xss[len(xss)-1].ts.Values[i] * phi vReq := xss[len(xss)-1].ts.Values[i] * phi
if math.IsNaN(vReq) || vReq < 0 {
// Broken bucket.
return nan
}
for _, xs := range xss { for _, xs := range xss {
v := xs.ts.Values[i] v := xs.ts.Values[i]
if math.IsNaN(v) || v < 0 {
// Broken bucket.
return nan
}
le := xs.le le := xs.le
if v < vReq { if v < vReq {
vPrev = v vPrev = v