From 614af10991c395cc7b057ed29cec93af293c2ce7 Mon Sep 17 00:00:00 2001 From: sandeep6189 Date: Wed, 10 Apr 2024 09:56:47 -0700 Subject: [PATCH] fix(query): Reverting the min max suffix replacement (#1757) --- .../scala/filodb/query/exec/MultiSchemaPartitionsExec.scala | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/query/src/main/scala/filodb/query/exec/MultiSchemaPartitionsExec.scala b/query/src/main/scala/filodb/query/exec/MultiSchemaPartitionsExec.scala index 6b9b7d8c6..332c566b1 100644 --- a/query/src/main/scala/filodb/query/exec/MultiSchemaPartitionsExec.scala +++ b/query/src/main/scala/filodb/query/exec/MultiSchemaPartitionsExec.scala @@ -67,7 +67,7 @@ final case class MultiSchemaPartitionsExec(queryContext: QueryContext, /* * As part of Histogram query compatibility with Prometheus format histograms, we - * remove _sum, _count, _min, _max suffix from metric name here. _bucket & le are already + * remove _sum, _count suffix from metric name here. _bucket & le are already * removed in SingleClusterPlanner. We remove the suffix only when partition lookup does not return any results */ if (lookupRes.firstSchemaId.isEmpty && querySession.queryConfig.translatePromToFilodbHistogram && @@ -76,10 +76,6 @@ final case class MultiSchemaPartitionsExec(queryContext: QueryContext, removeSuffixAndGenerateLookupResult(filters, metricName.get, "sum", source, querySession) else if (metricName.get.endsWith("_count")) removeSuffixAndGenerateLookupResult(filters, metricName.get, "count", source, querySession) - else if (metricName.get.endsWith("_min")) - removeSuffixAndGenerateLookupResult(filters, metricName.get, "min", source, querySession) - else if (metricName.get.endsWith("_max")) - removeSuffixAndGenerateLookupResult(filters, metricName.get, "max", source, querySession) else (lookupRes, newColName) lookupRes = res._1