alamb commented on PR #3690:
URL: https://github.com/apache/arrow-rs/pull/3690#issuecomment-1456724276

   > it trades a behaviour change that results in an error, to a behaviour 
change that results in silent truncation? Am I missing something here?
   
   I think another concern is that this trade comes with a significant 
performance cost for the non overflow case
   
   What if we changed the DataFusion type coercion code so that it was able to 
handle the large literal cases 
   
   Like special cased
   
https://github.com/apache/arrow-datafusion/blob/99ef989312292b04efdf06b178617e9008b46c84/datafusion/optimizer/src/type_coercion.rs#L589-L598
   
   So that it could handle a literal that was too large for its declared type?
   


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