gianm commented on a change in pull request #10198:
URL: https://github.com/apache/druid/pull/10198#discussion_r459039550
##########
File path: docs/querying/sql.md
##########
@@ -629,7 +629,9 @@ Druid SQL uses four different native query types.
period)`, have no other grouping expressions, no HAVING or LIMIT clauses, no
nesting, and either no ORDER BY, or an
ORDER BY that orders by same expression as present in GROUP BY. It also uses
Timeseries for "grand total" queries that
have aggregation functions but no GROUP BY. This query type takes advantage of
the fact that Druid segments are sorted
-by time.
+by time. The SQL planner can inject `timestampResultField` in the query
context when there is a grouping key. This
Review comment:
Btw, the rest of this patch LGTM, other than the doc stuff mentioned
here.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
