Skip to content
This repository was archived by the owner on Aug 18, 2025. It is now read-only.

Commit b2af85e

Browse files
authored
Merge pull request #79 from v3io/development
ignore step&aggrWindow when tsdb should not do any aggregations (#78)
2 parents 8d3e0b7 + c20e3a4 commit b2af85e

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

storage/tsdb/promtsdb.go

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,13 +140,20 @@ func (promQuery *V3ioPromQuerier) Select(params *storage.SelectParams, oms ...*l
140140
promQuery.LastTSDBAggregatedAggr = params.Func
141141
}
142142

143+
// In case we can not do aggregations, make sure no step or aggregation window is passed.
144+
step,aggrWindow := params.Step, params.AggregationWindow
145+
if !promQuery.UseV3ioAggregations(){
146+
step = 0
147+
aggrWindow = 0
148+
}
149+
143150
selectParams := &pquerier.SelectParams{Name: name,
144151
Functions: function,
145-
Step: params.Step,
152+
Step: step,
146153
Filter: filter,
147154
From: promQuery.mint,
148155
To: promQuery.maxt,
149-
AggregationWindow: params.AggregationWindow}
156+
AggregationWindow: aggrWindow}
150157

151158
promQuery.logger.DebugWith("Going to query tsdb", "params", selectParams,
152159
"UseAggregates", promQuery.UseAggregates, "UseAggregatesConfig", promQuery.UseAggregatesConfig)

0 commit comments

Comments
 (0)