Skip to content

fix: Prevent Coordinator startup failure when enabling centralizedDatasourceSchema#19265

Merged
gianm merged 1 commit intoapache:masterfrom
GWphua:segmentmetadata-coordinator
Apr 8, 2026
Merged

fix: Prevent Coordinator startup failure when enabling centralizedDatasourceSchema#19265
gianm merged 1 commit intoapache:masterfrom
GWphua:segmentmetadata-coordinator

Conversation

@GWphua
Copy link
Copy Markdown
Contributor

@GWphua GWphua commented Apr 6, 2026

Coordinator fails to start when users set druid.centralizedDatasourceSchema.enabled=true

Logs:


1) [Guice/MissingImplementation]: No implementation for QueryConfigProvider was bound.

Requested by:
1  : QueryLifecycleFactory.<init>(QueryLifecycleFactory.java:66)
      \_ for 9th parameter
     at CoordinatorSegmentMetadataCache.<init>(CoordinatorSegmentMetadataCache.java:144)
      \_ for 1st parameter
     at DruidCoordinator.<init>(DruidCoordinator.java:198)
      \_ for 14th parameter
     at CliCoordinator$1.configure(CliCoordinator.java:223)
      \_ installed by: Modules$OverrideModule -> Modules$OverrideModule -> CliCoordinator$1

Learn more:
  https://github.com/google/guice/wiki/MISSING_IMPLEMENTATION

1 error

======================
Full classname legend:
======================
CliCoordinator$1:                "org.apache.druid.cli.CliCoordinator$1"
CoordinatorSegmentMetadataCache: "org.apache.druid.segment.metadata.CoordinatorSegmentMetadataCache"
DruidCoordinator:                "org.apache.druid.server.coordinator.DruidCoordinator"
Modules$OverrideModule:          "com.google.inject.util.Modules$OverrideModule"
QueryConfigProvider:             "org.apache.druid.query.QueryConfigProvider"
QueryLifecycleFactory:           "org.apache.druid.server.QueryLifecycleFactory"
========================
End of classname legend:
========================

	at com.google.inject.internal.Errors.throwCreationExceptionIfErrorsExist(Errors.java:589)
	at com.google.inject.internal.InternalInjectorCreator.initializeStatically(InternalInjectorCreator.java:163)
	at com.google.inject.internal.InternalInjectorCreator.build(InternalInjectorCreator.java:110)
	at com.google.inject.Guice.createInjector(Guice.java:87)
	at com.google.inject.Guice.createInjector(Guice.java:69)
	at com.google.inject.Guice.createInjector(Guice.java:59)
	at org.apache.druid.initialization.ExtensionInjectorBuilder.build(ExtensionInjectorBuilder.java:49)
	at org.apache.druid.initialization.ServerInjectorBuilder.build(ServerInjectorBuilder.java:118)
	at org.apache.druid.initialization.ServerInjectorBuilder.makeServerInjector(ServerInjectorBuilder.java:73)
	at org.apache.druid.cli.GuiceRunnable.makeInjector(GuiceRunnable.java:85)
	... 2 more

Release note

Fix an issue where Coordinator cannot start when druid.centralizedDatasourceSchema.enabled=true


This PR has:

  • been self-reviewed.
  • been tested in a test Druid cluster.

@GWphua GWphua changed the title Prevent Coordinator startup failure when enabling centralizedDatasourceSchema fix: Prevent Coordinator startup failure when enabling centralizedDatasourceSchema Apr 6, 2026
@GWphua
Copy link
Copy Markdown
Contributor Author

GWphua commented Apr 7, 2026

Adding jacoco, SegmentSchemaCacheModule has no existing binding tests. Please lmk if you feel one is needed.

@GWphua GWphua closed this Apr 7, 2026
@GWphua GWphua reopened this Apr 7, 2026
@GWphua GWphua added this to the 37.0.0 milestone Apr 8, 2026
@gianm gianm merged commit 8a81601 into apache:master Apr 8, 2026
127 of 130 checks passed
@GWphua GWphua deleted the segmentmetadata-coordinator branch April 8, 2026 06:42
cecemei pushed a commit to cecemei/druid that referenced this pull request Apr 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants