Conversation
There was a problem hiding this comment.
I think it'd be good to leave the header but then in the body of this section, deep link to the relevant extension doc. That way it's clear that we support HDFS/S3 and there are not too many clicks needed to get there.
There was a problem hiding this comment.
Similar comment for MD stores
|
👍 |
There was a problem hiding this comment.
"See the [druid-s3-extensions extension documentation]." would read better than "See [here]."
|
this change was discussed during the dev sync on 2016-03-22 as a way of clarifying which configs are from extensions and which configs are not. 👍 |
|
this change moved some lookup docs into the wrong directory. |
|
@drcrallen which ones? Will you be opening up a PR to fix that or should @fjy look into it? |
|
Dynamic Config is not an extension as it does not actually DO anything, but rather just provides a framework for getting the config around. There's the part that's in druid as of 0.9.1 #2735 |
4 participants
No description provided.