gianm commented on a change in pull request #11549:
URL: https://github.com/apache/druid/pull/11549#discussion_r686977832
##########
File path: docs/querying/segmentmetadataquery.md
##########
@@ -144,16 +144,16 @@ Types of column analyses are described below:
### cardinality
-* `cardinality` in the result will return the size of the bitmap index or
dictionary encoding for string dimensions, or null for other dimension types.
- If `merge` was set, the result will be the max of this value across segments.
Only relevant for dimension columns.
+* `cardinality` in the result will return the number of unique values present
in a string column. It is null for other column types.
+ If `merge` is set, the result will be the max of this value across segments.
Only relevant for string columns.
### minmax
-* Estimated min/max values for each column. Only relevant for dimension
columns.
+* Estimated min/max values for each column. Only reported for string columns.
### size
-* `size` in the result will contain the estimated total segment byte size as
if the data were stored in text format
+* `size` in the result will contain the estimated total segment byte size as
if the data were stored in text format. This is _not_ the actual storage size
of the column in Druid.
Review comment:
I pushed some new content that hopefully clears this up. Please let me
know if it makes sense to you.
##########
File path: docs/querying/segmentmetadataquery.md
##########
@@ -144,16 +144,16 @@ Types of column analyses are described below:
### cardinality
-* `cardinality` in the result will return the size of the bitmap index or
dictionary encoding for string dimensions, or null for other dimension types.
- If `merge` was set, the result will be the max of this value across segments.
Only relevant for dimension columns.
+* `cardinality` in the result will return the number of unique values present
in a string column. It is null for other column types.
+ If `merge` is set, the result will be the max of this value across segments.
Only relevant for string columns.
Review comment:
I pushed some new content that hopefully clears this up. Please let me
know if it makes sense to you.
--
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]
