From f4c73e149984ea25113e204b2aae0abd811334f5 Mon Sep 17 00:00:00 2001 From: Zoltan Haindrich Date: Thu, 16 May 2024 11:31:28 +0000 Subject: [PATCH] make old defaults to overides --- .../druid/sql/calcite/util/SqlTestFramework.java | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/sql/src/test/java/org/apache/druid/sql/calcite/util/SqlTestFramework.java b/sql/src/test/java/org/apache/druid/sql/calcite/util/SqlTestFramework.java index 74cb4dbdaa2..d4b6026d8f4 100644 --- a/sql/src/test/java/org/apache/druid/sql/calcite/util/SqlTestFramework.java +++ b/sql/src/test/java/org/apache/druid/sql/calcite/util/SqlTestFramework.java @@ -654,14 +654,20 @@ return componentSupplier.getPlannerComponentSupplier().createViewManager(); // test pulls in a module, then pull in that module, even though we are // not the Druid node to which the module is scoped. .ignoreLoadScopes() - .addModule(new LookylooModule()) - .addModule(new SegmentWranglerModule()) - .addModule(new SqlAggregationModule()) +// .addModule(new LookylooModule()) +// .addModule(new SegmentWranglerModule()) +// .addModule(new SqlAggregationModule()) // .addModule(new SqlModule()) - .addModule(new ExpressionModule()); +// .addModule(new ExpressionModule()); // .addModule(testSetupModule()); + ; List overrideModules = new ArrayList<>(builder.overrideModules); + overrideModules.add(new LookylooModule()); + overrideModules.add(new SqlAggregationModule()); + overrideModules.add(new SegmentWranglerModule()); + overrideModules.add(new ExpressionModule()); + overrideModules.add(testSetupModule()); builder.componentSupplier.configureGuice(injectorBuilder, overrideModules);