From 0ffd391f1e195c00e5e6be84037604e30f006d67 Mon Sep 17 00:00:00 2001 From: Alex Theimer Date: Fri, 12 Jan 2024 12:02:38 -0800 Subject: [PATCH] Revert "DROP: printPairs" This reverts commit 2054a7ff87eefd6acfcf1100f4ec198de3368d2e. --- .../queryplanner/PlannerHierarchySpec.scala | 15 --------------- 1 file changed, 15 deletions(-) diff --git a/coordinator/src/test/scala/filodb.coordinator/queryplanner/PlannerHierarchySpec.scala b/coordinator/src/test/scala/filodb.coordinator/queryplanner/PlannerHierarchySpec.scala index bd38467827..5b2c9a55be 100644 --- a/coordinator/src/test/scala/filodb.coordinator/queryplanner/PlannerHierarchySpec.scala +++ b/coordinator/src/test/scala/filodb.coordinator/queryplanner/PlannerHierarchySpec.scala @@ -18,9 +18,6 @@ import filodb.prometheus.parse.Parser.Antlr import filodb.query.{BadQueryException, IntervalSelector, LabelCardinality, PlanValidationSpec, RawSeries} import filodb.query.exec._ - -import scala.collection.mutable - // scalastyle:off line.size.limit // scalastyle:off number.of.methods class PlannerHierarchySpec extends AnyFunSpec with Matchers with PlanValidationSpec { @@ -593,18 +590,6 @@ class PlannerHierarchySpec extends AnyFunSpec with Matchers with PlanValidationS validatePlan(execPlan, expected) } - class QueryExpectedPairs { - val pairs = new mutable.ArrayBuffer[(String, String)] - def addPair(query: String, plan: ExecPlan): Unit = { - pairs.append((query, plan.printTree().replaceAll("\n", "\n |"))) - } - def printPairs(): Unit = { - for ((query, expected) <- pairs) { - println(s"""(\"\"\"$query\"\"\",\n \"\"\"$expected\"\"\".stripMargin),""") - } - } - } - it("should not pushdown root scalar operation into a RemoteExec's promql when query spans multiple partitions") { val queryExpectedPairs = Seq(