Skip to content

Commit

Permalink
remove TestSegmentsOverseer for now
Browse files Browse the repository at this point in the history
  • Loading branch information
kgyrtkirk committed Dec 19, 2024
1 parent 613bb56 commit c74559d
Showing 1 changed file with 7 additions and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -755,6 +740,13 @@ ServiceEmitter getServiceEmitter()
return NoopServiceEmitter.instance();
}

@Provides
@LazySingleton
public QuerySegmentWalker getQuerySegmentWalker(SpecificSegmentsQuerySegmentWalker walker)
{
return walker;
}

@Provides
GenericQueryMetricsFactory getGenericQueryMetricsFactory()
{
Expand Down Expand Up @@ -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);

Expand Down

0 comments on commit c74559d

Please sign in to comment.