clintropolis commented on a change in pull request #7082: Register lookup
module with overlord and middle manager.
URL: https://github.com/apache/incubator-druid/pull/7082#discussion_r264888771
##########
File path: services/src/main/java/org/apache/druid/cli/CliMiddleManager.java
##########
@@ -163,7 +164,8 @@ public WorkerNodeService getWorkerNodeService(WorkerConfig
workerConfig)
}
},
new IndexingServiceFirehoseModule(),
- new IndexingServiceTaskLogsModule()
+ new IndexingServiceTaskLogsModule(),
+ new LookupModule()
Review comment:
#7222 introduces `LookupSerdeModule` which I think might be nicer behavior
for middle manager and overlord than using `LookupModule` and configuring it
not to load, but it will require this PR to wait until the other is merged.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
