[GitHub] [arrow-rs] tustvold commented on issue #4505: No check when casting Int64 to Timestamp(Millisecond/Second).

["tustvold (via GitHub)" <gi...@apache.org>]

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

   We used to do something similar for decimals, constraining the values as per the precision, however, this added non-trivial overheads to many kernels and made the types difficult to safely use without repeatedly revalidating data.
   
   There is an issue where sometimes out of range data results in panics, we should fix this and make them return an error, would this be sufficient?


-- 
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