kumarUjjawal commented on PR #22057:
URL: https://github.com/apache/datafusion/pull/22057#issuecomment-4399416884

    I am not in favor of this PR. For the default/non-ANSI mode, the current 
behavior already matches Spark: `make_interval` is nullable and overflow 
returns NULL. 
   
   The ANSI-mode part needs a broader design discussion because it changes core 
UDF return-field API and still does not prove the full SQL/planning path works 
correctly.
   
   Given that, I think we should close this PR for now instead of making this 
change.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to