ektravel commented on code in PR #12549:
URL: https://github.com/apache/druid/pull/12549#discussion_r1117546410


##########
docs/querying/sql-data-types.md:
##########
@@ -33,7 +33,7 @@ Columns in Druid are associated with a specific data type. 
This topic describes
 
 Druid natively supports five basic column types: "long" (64 bit signed int), 
"float" (32 bit float), "double" (64 bit
 float) "string" (UTF-8 encoded strings and string arrays), and "complex" 
(catch-all for more exotic data types like

Review Comment:
   ```suggestion
   Druid natively supports the following column types: 
   - "long": 64-bit signed int
   - "float": 32-bit float
   - "double": 64-bit float
   - "string": UTF-8 encoded strings and string arrays
   - "complex": non-standard data types, such as nested JSON, hyperUnique and 
approxHistogram aggregators, and DataSketches aggregators
    - "array": array of any supported column types
   ```
   Consider presenting this information as a list to improve readability. 



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to