From 06aa16a34c117bdd34a647c2d4c992472bc251ff Mon Sep 17 00:00:00 2001 From: jakenichols2719 Date: Thu, 20 Oct 2022 11:59:49 -0600 Subject: [PATCH] InfluxDB ParseTimeRangeQuery() now returns _, _, true, _ if non-ts request is SELECT and can be cached Signed-off-by: jakenichols2719 --- pkg/backends/influxdb/handler_query.go | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/pkg/backends/influxdb/handler_query.go b/pkg/backends/influxdb/handler_query.go index 44124003f..028a127b5 100644 --- a/pkg/backends/influxdb/handler_query.go +++ b/pkg/backends/influxdb/handler_query.go @@ -95,11 +95,14 @@ func (c *Client) ParseTimeRangeQuery(r *http.Request) (*timeseries.TimeRangeQuer trq.Step = -1 var hasTimeQueryParts bool statements := make([]string, 0, len(q.Statements)) + var canObjectCache bool var cacheError error for _, v := range q.Statements { sel, ok := v.(*influxql.SelectStatement) if !ok || sel.Condition == nil { cacheError = errors.ErrNotTimeRangeQuery + } else { + canObjectCache = true } step, err := sel.GroupByInterval() if err != nil { @@ -159,6 +162,6 @@ func (c *Client) ParseTimeRangeQuery(r *http.Request) (*timeseries.TimeRangeQuer return trq, rlo, true, cacheError } - return trq, rlo, false, nil + return trq, rlo, canObjectCache, nil }