app/vmselect: unconditionally offset -step to rollup_candlestick. This makes results more consistent

This commit is contained in:
Aliaksandr Valialkin 2020-02-04 23:31:06 +02:00
parent ce38b176bc
commit 8e77b54846

View File

@ -435,7 +435,7 @@ func evalRollupFunc(ec *EvalConfig, name string, rf rollupFunc, expr metricsql.E
ecNew.End = end ecNew.End = end
} }
} }
if name == "rollup_candlestick" && len(re.Window) == 0 { if name == "rollup_candlestick" {
// Automatically apply `offset -step` to `rollup_candlestick` function // Automatically apply `offset -step` to `rollup_candlestick` function
// in order to obtain expected OHLC results. // in order to obtain expected OHLC results.
// See https://github.com/VictoriaMetrics/VictoriaMetrics/issues/309#issuecomment-582113462 // See https://github.com/VictoriaMetrics/VictoriaMetrics/issues/309#issuecomment-582113462