tustvold commented on issue #5200:
URL: https://github.com/apache/arrow-rs/issues/5200#issuecomment-1850318111

   > AFAIK Postgre defines a month as exactly 30 days
   
   This was what I had understood as well. What do you think of updating the 
cast kernel to allow casting interval expressions with non-zero days/months to 
durations, using the same assumptions as postgres makes?
   
   This would have a number of quite compelling advantages in my opinion:
   
   * The conversion is explicit and user-visible
   * The ordering behaviour for durations is easy to understand
   * No additional kernels required
   
   My main hesitancy with defining our own custom partial ordering is that I am 
not very confident of all the nuances involved in performing such an operation 
correctly, how to communicate these to users, and am concerned about adding 
further edge-cases for people to be wary of. Whilst I agree the postgres 
approach is making some pretty crude assumptions, it is easy to understand and 
comes with prior precedent...
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to