[GitHub] [arrow-datafusion] alamb opened a new issue #200: Add support for AVG(Timestamp) types

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

alamb opened a new issue #200:
URL: https://github.com/apache/arrow-datafusion/issues/200


   *Note*: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-12318
   
   This is a follow on to ARROW-12277
   
   Background: Support for Min/Max/Sum/Count were added for DataType::Timestamp(*) types in https://github.com/apache/arrow/pull/9970.
   
   This ticket tracks adding support for Avg, which is slightly more involved as currently Avg assumes the output type is always F64, and in this case I think Avg(timestamp) should also be (timestamp). We should double check what postgres does in this case and follow its example
   


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

[GitHub] [arrow-datafusion] alamb commented on issue #200: Add support for AVG(Timestamp) types

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

alamb commented on issue #200:
URL: https://github.com/apache/arrow-datafusion/issues/200#issuecomment-826839478


   Comment from Andrew Lamb(alamb) @ 2021-04-11T09:48:09.161+0000:
   <pre>[~Dandandan] notes that PostgreSQL doesn't support SUM or AVG for timestamps: https://www.postgresql.org/docs/13/functions-aggregate.html
   
   
   so perhaps we should not support it in DataFusion either :thinking_face:</pre>


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