[GitHub] [druid] vogievetsky commented on a diff in pull request #14065: docs: add docs for schema auto-discovery

["vogievetsky (via GitHub)" <gi...@apache.org>]

vogievetsky commented on code in PR #14065:
URL: https://github.com/apache/druid/pull/14065#discussion_r1186922589


##########
docs/ingestion/schema-design.md:
##########
@@ -241,12 +240,45 @@ You should query for the number of ingested rows with:
 ]
 ```
 
-### Schema-less dimensions
+### Schema auto-discovery for dimensions
+
+Druid can infer the schema for your data in one of two ways:
+
+- [Type-aware schema discovery (experimental)](#type-aware-schema-discovery-experimental) where Druid infers the schema and type for your data. Type-aware schema discovery is experimental currently available for native batch and streaming ingestion.
+- [String-based schema discovery](#string-based-schema-discovery) where all the discovered columns are typed as either native string or multi-value string columns.
+
+#### Type-aware schema discovery (experimental)

Review Comment:
   I think it is better not to put `(experimental)` in the heading itself as later, when we remove it we will have to remember to add a redirect. Better to just mention it in the warning text below



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

To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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