[GitHub] [incubator-druid] jon-wei commented on a change in pull request #8872: Improve missing JDBC driver error for lookups

[GitBox <gi...@apache.org>]

jon-wei commented on a change in pull request #8872: Improve missing JDBC driver error for lookups
URL: https://github.com/apache/incubator-druid/pull/8872#discussion_r347570922
 
 

 ##########
 File path: docs/development/extensions-core/druid-lookups.md
 ##########
 @@ -33,6 +33,12 @@ This module can be used side to side with other lookup module like the global ca
 
 To use this extension please make sure to  [include](../../development/extensions.md#loading-extensions) `druid-lookups-cached-single` as an extension.
 
+> If using JDBC, you will need to add your database's client JAR files to the extension's directory.
 
 Review comment:
   Can you add a similar note to `lookups-cached-global.md`?

----------------------------------------------------------------
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