techdocsmith commented on a change in pull request #11672:
URL: https://github.com/apache/druid/pull/11672#discussion_r706291602



##########
File path: docs/querying/granularities.md
##########
@@ -28,9 +28,11 @@ sidebar_label: "Granularities"
 > language. For information about time functions available in SQL, refer to the
 > [SQL documentation](sql.md#time-functions).
 
-The granularity field determines how data gets bucketed across the time 
dimension, or how it gets aggregated by hour, day, minute, etc.
+The granularity determines how data gets bucketed across the time dimension, 
or how it gets aggregated by hour, day, minute, etc.
 
-It can be specified either as a string for simple granularities or as an 
object for arbitrary granularities.
+For example, use time granularities in [native queries](querying.md) to bucket 
results by time, and in the `dataSchema` \\ 
[`granularitySpec`](../ingestion/ingestion-spec.md#granularityspec) section of 
ingestion specifications to segment incoming data.

Review comment:
       Thanks @petermarshallio . I agree with you that this is a "common 
concept" since it's shared between native queries and ingestion. There's some 
room for improvement in the overall architecture here.
   Very much appreciate your POV and all the work you are doing to improve the 
Apache Druid docs!




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