Kiu/VictoriaMetrics
Kiu/VictoriaMetrics
1
0
Fork 0
mirror of https://github.com/VictoriaMetrics/VictoriaMetrics.git synced 2024-12-15 00:13:30 +01:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

app/vmselect/promql: increase the estimated number of time series returned by aggr() by (something) from 100 to 1K, since 100 may result in OOM for high number of time series

Browse Source
This commit is contained in:
Aliaksandr Valialkin 2020-01-03 01:02:04 +02:00
parent e0abf45d45
commit 6ea7f23446
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/vmselect/promql/eval.go
View File

@ -628,8 +628,8 @@ func evalRollupFuncWithMetricExpr(ec *EvalConfig, name string, rf rollupFunc, me
if iafc.ae.Modifier.Op != "" { if iafc.ae.Modifier.Op != "" {
// Increase the number of timeseries for non-empty group list: `aggr() by (something)`, // Increase the number of timeseries for non-empty group list: `aggr() by (something)`,
// since each group can have own set of time series in memory. // since each group can have own set of time series in memory.
// Estimate the number of such groups is lower than 100 :) // Estimate the number of such groups is lower than 1000 :)
timeseriesLen *= 100 timeseriesLen *= 1000
} }
} }
rollupPoints := mulNoOverflow(pointsPerTimeseries, int64(timeseriesLen*len(rcs))) rollupPoints := mulNoOverflow(pointsPerTimeseries, int64(timeseriesLen*len(rcs)))