You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/07/15 21:29:50 UTC

[GitHub] [arrow-datafusion] andygrove opened a new issue, #2922: Support `mean` as synonym for `avg`

andygrove opened a new issue, #2922:
URL: https://github.com/apache/arrow-datafusion/issues/2922

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   This came up in https://github.com/datafusion-contrib/datafusion-python/issues/56
   
   **Describe the solution you'd like**
   Support "mean" as well as "avg"
   
   **Describe alternatives you've considered**
   None
   
   **Additional context**
   None
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow-datafusion] BaymaxHWY commented on issue #2922: Support `mean` as synonym for `avg`

Posted by GitBox <gi...@apache.org>.
BaymaxHWY commented on issue #2922:
URL: https://github.com/apache/arrow-datafusion/issues/2922#issuecomment-1186072087

   I'm interested in this. May i try it?


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-datafusion] andygrove closed issue #2922: Support `mean` as synonym for `avg`

Posted by GitBox <gi...@apache.org>.
andygrove closed issue #2922: Support `mean` as synonym for `avg`
URL: https://github.com/apache/arrow-datafusion/issues/2922


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-datafusion] andygrove commented on issue #2922: Support `mean` as synonym for `avg`

Posted by GitBox <gi...@apache.org>.
andygrove commented on issue #2922:
URL: https://github.com/apache/arrow-datafusion/issues/2922#issuecomment-1185959157

   If anyone is interested in picking this up, it looks like it is probably a one liner (plus tests).
   
   We need to add an additional mapping here in `datafusion/expr/src/aggregate_function.rs`
   
   ```
   impl FromStr for AggregateFunction {
       type Err = DataFusionError;
       fn from_str(name: &str) -> Result<AggregateFunction> {
           Ok(match name {
               "min" => AggregateFunction::Min,
               "max" => AggregateFunction::Max,
               "count" => AggregateFunction::Count,
               "avg" => AggregateFunction::Avg,
   ```


-- 
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: github-unsubscribe@arrow.apache.org

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