kasakrisz commented on PR #5091: URL: https://github.com/apache/hive/pull/5091#issuecomment-2089999923
>We will make the UDFs skip invalid texts and SUM('text') return NULL. Maybe we will fix GenericUDAFSum. Yes, it seems reasonable because Hive doesn't follow the standard in the case of `cast` function either: Cast returns NULL instead of throwing exception due to the reasons Aman mentioned. I think we can consider `sum(string)` as an overload of the standard `sum` function which should behave like `sum(cast('text' as <numeric_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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org