suneet-s commented on a change in pull request #9122: Add SQL GROUPING SETS 
support.
URL: https://github.com/apache/druid/pull/9122#discussion_r384134632
 
 

 ##########
 File path: docs/querying/sql.md
 ##########
 @@ -65,7 +65,7 @@ Druid SQL supports SELECT queries with the following 
structure:
 SELECT [ ALL | DISTINCT ] { * | exprs }
 FROM table
 [ WHERE expr ]
-[ GROUP BY exprs ]
+[ GROUP BY [ exprs | GROUPING SETS ( (exprs), ... ) | ROLLUP (exprs) | CUBE 
(exprs) ] ]
 
 Review comment:
   I was thrown off by this in the docs 
   
   `[ EXPLAIN PLAN FOR ]
   [ WITH tableName [ ( column1, column2, ... ) ] AS ( query ) ]`
   
   I didn't realize the parenthesis were required in this statement . I 
couldn't get an EXPLAIN query to work locally for me.

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


With regards,
Apache Git Services

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

Reply via email to