[GitHub] [druid] clintropolis commented on a change in pull request #12093: Docs: add FILTER to sql query syntax

[GitBox <gi...@apache.org>]

clintropolis commented on a change in pull request #12093:
URL: https://github.com/apache/druid/pull/12093#discussion_r774246307



##########
File path: docs/querying/sql.md
##########
@@ -49,6 +49,7 @@ Druid SQL supports SELECT queries with the following structure:
 [ WITH tableName [ ( column1, column2, ... ) ] AS ( query ) ]
 SELECT [ ALL | DISTINCT ] { * | exprs }
 FROM { <table> | (<subquery>) | <o1> [ INNER | LEFT ] JOIN <o2> ON condition }
+[ FILTER ( WHERE condition ) ]

Review comment:
       this is an optional part of SELECT and also HAVING, so I guess more accurate would be removing this newly added line and changing those lines to:
   
   ```
   SELECT [ ALL | DISTINCT ] { * | exprs } [ FILTER ( WHERE expr ) ]
   ```
   and
   ```
   [ HAVING expr [ FILTER ( WHERE expr ) ] ]
   ```
   I think?




-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org