gianm commented on a change in pull request #7591: Remove SQL experimental
banner and other doc adjustments.
URL: https://github.com/apache/incubator-druid/pull/7591#discussion_r280925271
##########
File path: docs/content/toc.md
##########
@@ -164,10 +165,8 @@ layout: toc
* Experimental Features
* [Overview](/docs/VERSION/development/experimental.html)
* [Approximate Histograms and
Quantiles](/docs/VERSION/development/extensions-core/approximate-histograms.html)
- *
[Datasketches](/docs/VERSION/development/extensions-core/datasketches-extension.html)
* [Geographic Queries](/docs/VERSION/development/geo.html)
* [Router](/docs/VERSION/development/router.html)
- * [Kafka Indexing
Service](/docs/VERSION/development/extensions-core/kafka-ingestion.html)
Review comment:
KIS is already in the ingestion section.
DataSketches isn't in the ToC but it's described in the approximate
algorithms and sql sections.
Actually, now that I think about this a bit more, I think removing the
entire experimental feature section is a good idea. There is no particular
reason to have them displayed so prominently. It is better for them to be in
the sections that make sense for them (node type, aggregator, whatever).
----------------------------------------------------------------
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]
