maropu commented on a change in pull request #29056:
URL: https://github.com/apache/spark/pull/29056#discussion_r456986616



##########
File path: docs/sql-ref-syntax-qry-select-groupby.md
##########
@@ -91,6 +93,18 @@ aggregate_name ( [ DISTINCT ] expression [ , ... ] ) [ 
FILTER ( WHERE boolean_ex
     Filters the input rows for which the `boolean_expression` in the `WHERE` 
clause evaluates
     to true are passed to the aggregate function; other rows are discarded.
 
+* **FIRST**
+
+    `FIRST` selects a first expression value from the data set. We can specify 
an optional `IGNORE NULL` clause to ignore NULL values.
+    
+* **LAST**
+
+    `LAST` selects a last expression value from the data set. We can specify 
an optional `IGNORE NULLS` clause to ignore NULL values. 
+    
+* **IGNORE NULLS**
+   
+   `IGNORE NULLS` is used to ignore null values, which are used in `FIRST` and 
`LAST` 
+

Review comment:
       Looks verbose, so could you remove this part?




----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to