dclim commented on a change in pull request #6122: New docs intro URL: https://github.com/apache/incubator-druid/pull/6122#discussion_r208717704
########## File path: docs/content/toc.md ########## @@ -3,7 +3,12 @@ layout: toc --- ## Getting Started - * [Concepts](/docs/VERSION/design/) + * [Design](/docs/VERSION/design/index.html) Review comment: Maybe nit-picky, but I would expect all sections with the same header level to be listed here. I also think 'What is Druid?' should be the same header level as 'Architecture', 'Query processing', etc. (right now it's level 1 and the rest are level 2). So I think this should read: What is Druid? When should I use Druid? Architecture Datasources and segments Query processing External dependencies I'm indifferent about 'Ingestion overview' being repeated here and in the 'Data Ingestion' section. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on 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]
