Add support for AVG(Timestamp) types #200

Activity

1. alamb

   added

   datafusion

   on Apr 26, 2021
2. alamb commented on Apr 26, 2021

   

   alamb

   on Apr 26, 2021

   ContributorAuthor

   More actions

   Comment from Andrew Lamb(alamb) @ 2021-04-11T09:48:09.161+0000:

   [~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:
3. waitingkuo commented on Aug 15, 2022

   

   waitingkuo

   on Aug 15, 2022

   Contributor

   More actions

   @alamb

   Postgresql has AVG and SUM for time but not timestamps

   Should we close this and submit another ticket for AVG(Time) and SUM(Time)?
4. alamb commented on Aug 15, 2022

   

   alamb

   on Aug 15, 2022

   ContributorAuthor

   More actions

   @waitingkuo -- I think that sounds like a good idea. I will do so.

   👍React with 👍1Reacted by Wei-Ting Kuo
5. alamb

   closed this as completedon Aug 15, 2022
6. waitingkuo commented on Aug 15, 2022

   

   waitingkuo

   on Aug 15, 2022

   Contributor

   More actions

   @alamb i just tested postgrseql, looks like SUM and AVG cast Time to Interval and then SUM / AVG them

   willy=# select pg_typeof(SUM(time '00:00:00')),  pg_typeof(AVg(time '00:00:00'));
    pg_typeof | pg_typeof 
   -----------+-----------
    interval  | interval
   (1 row)

   this make sense otherwise SUM(Time) is just quite strange
7. alamb commented on Aug 15, 2022

   

   alamb

   on Aug 15, 2022

   ContributorAuthor

   More actions

   #3166 👍
8. HairstonE

   added a commit that references this issue on Oct 7, 2025

   Run `maturin develop` instead of `cargo build` in verification script (…

   ...

   bb004ee