From c74559dadc85b5514e3d6ca47ed62cd459bf20d7 Mon Sep 17 00:00:00 2001
From: Zoltan Haindrich <kirk@rxd.hu>
Date: Thu, 19 Dec 2024 12:17:45 +0000
Subject: [PATCH] remove TestSegmentsOverseer for now

---
 .../sql/calcite/util/SqlTestFramework.java    | 23 ++++++-------------
 1 file changed, 7 insertions(+), 16 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 584d9d207aec..b8656cc777ec 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
@@ -706,21 +706,6 @@ public SqlStatementFactory statementFactory()
     }
   }
 
-  static class TestSegmentsOverseer implements DruidModule
-  {
-    @Override
-    public void configure(Binder binder)
-    {
-    }
-
-    @Provides
-    @LazySingleton
-    public QuerySegmentWalker getQuerySegmentWalker(SpecificSegmentsQuerySegmentWalker walker)
-    {
-      return walker;
-    }
-  }
-
   /**
    * Guice module to create the various query framework items. By creating items within
    * a module, later items can depend on those created earlier by grabbing them from the
@@ -755,6 +740,13 @@ ServiceEmitter getServiceEmitter()
       return NoopServiceEmitter.instance();
     }
 
+    @Provides
+    @LazySingleton
+    public QuerySegmentWalker getQuerySegmentWalker(SpecificSegmentsQuerySegmentWalker walker)
+    {
+      return walker;
+    }
+
     @Provides
     GenericQueryMetricsFactory getGenericQueryMetricsFactory()
     {
@@ -1007,7 +999,6 @@ private SqlTestFramework(Builder builder)
 
     injectorBuilder.add(componentSupplier.getCoreModule());
     overrideModules.add(new TestSetupModule(builder));
-    overrideModules.add(new TestSegmentsOverseer());
     overrideModules.add(componentSupplier.getOverrideModule());
     builder.componentSupplier.configureGuice(injectorBuilder, overrideModules);