From cbe0d0a421d8ec3f74a320fbd4bb380b5b79f061 Mon Sep 17 00:00:00 2001
From: Gian Merlino <gian@imply.io>
Date: Sun, 26 May 2019 15:14:50 -0700
Subject: [PATCH] Fix LookupSerdeModule double-binding in
 Coordinator-as-Overlord mode. (#7765)

Fixes #7762.
---
 .../src/main/java/org/apache/druid/cli/CliCoordinator.java    | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/services/src/main/java/org/apache/druid/cli/CliCoordinator.java b/services/src/main/java/org/apache/druid/cli/CliCoordinator.java
index 88245624093d..b4a9516af277 100644
--- a/services/src/main/java/org/apache/druid/cli/CliCoordinator.java
+++ b/services/src/main/java/org/apache/druid/cli/CliCoordinator.java
@@ -273,6 +273,10 @@ public LoadQueueTaskMaster getLoadQueueTaskMaster(
 
     if (beOverlord) {
       modules.addAll(new CliOverlord().getModules(false));
+    } else {
+      // Only add LookupSerdeModule if !beOverlord, since CliOverlord includes it, and having two copies causes
+      // the injector to get confused due to having multiple bindings for the same classes.
+      modules.add(new LookupSerdeModule());
     }
 
     return modules;