[GitHub] [pinot] agavra commented on pull request #10021: support SKEW_POP and KURTOSIS_POP aggregates

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

agavra commented on PR #10021:
URL: https://github.com/apache/pinot/pull/10021#issuecomment-1362220879

   @snleee 
   
   > Can we also add SKEWSAMP/KURTOSISSAMP if it's just the matter of dividing by n-1?
   
   I checked whether this is common or not, and it seems that sample skew/sample kurtosis aren't really used (e.g. presto doesn't define those two functions) so I'll go ahead and just rename the functions `SKEW` and `KURTOSIS`


-- 
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@pinot.apache.org

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


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