You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/04/20 00:58:39 UTC

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #7082: Register lookup module with overlord and middle manager.

jon-wei 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_r277118169
 
 

 ##########
 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:
   @scrawfor #7222 was merged, can you please rebase when you get a chance?

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org