You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2022/02/10 00:43:32 UTC

[GitHub] [spark] amaliujia edited a comment on pull request #34868: [SPARK-37614][SQL] Support ANSI Aggregate Function: regr_avgx & regr_avgy

amaliujia edited a comment on pull request #34868:
URL: https://github.com/apache/spark/pull/34868#issuecomment-1034362773


   @beliefer can you document the new functions in the PR description?
   
   Basically can you list
   1. the function signature (name and argument type)
   2. Argument valid input range. Maybe some examples
   3. Not valid argument inputs. Maybe some examples.
   4. other special cases if any.
   ?
   There are usage comment for each expression. We can also copy documentation above into those comment.


-- 
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: reviews-unsubscribe@spark.apache.org

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



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