mirror of
https://github.com/VictoriaMetrics/VictoriaMetrics.git
synced 2024-12-18 14:40:26 +01:00
f825a9de80
### Describe Your Changes
evalInstantRollup could have overreport the number of fetched series if
`offset` checks will result into retry. This change updates fetched
series only if these checks were successful.
It also adds a comment to another potential place of over-reporting
series fetched. It doesn't fix it, because it would require spending
extra resources on such a check, while discrepancy in seriesFetched
doesn't affect calculations in any way.
Probably related to
https://github.com/VictoriaMetrics/VictoriaMetrics/issues/7170
### Checklist
The following checks are **mandatory**:
- [x] My change adheres [VictoriaMetrics contributing
guidelines](https://docs.victoriametrics.com/contributing/).
Signed-off-by: hagen1778 <roman@victoriametrics.com>
(cherry picked from commit
|
||
|---|---|---|
| .. | ||
| active_queries.go | ||
| aggr_incremental_test.go | ||
| aggr_incremental.go | ||
| aggr_test.go | ||
| aggr.go | ||
| binary_op.go | ||
| eval_test.go | ||
| eval.go | ||
| exec_test.go | ||
| exec.go | ||
| memory_limiter_test.go | ||
| memory_limiter.go | ||
| parser.go | ||
| rollup_result_cache_test.go | ||
| rollup_result_cache.go | ||
| rollup_test.go | ||
| rollup_timing_test.go | ||
| rollup.go | ||
| timeseries_test.go | ||
| timeseries.go | ||
| transform_test.go | ||
| transform.go | ||
| tzdata.go | ||