Aggarwal-Raghav opened a new pull request, #6907: URL: https://github.com/apache/hive/pull/6907
### What changes were proposed in this pull request? This issue was observed during https://github.com/apache/hive/pull/6681/ - Updated Calcite query planning in `HiveFunctionHelper` to invoke the non-deprecated `GenericUDTF.initialize(StructObjectInspector)` instead of calling the `@Deprecated array overload GenericUDTF.initialize(ObjectInspector[])` ### Why are the changes needed? In GenericUDTF, the default base implementation of the deprecated method explicitly throws, `java.lang.IllegalStateException: Should not be called directly` However, `HiveFunctionHelper.getReturnType()` hardcoded a direct call to the deprecated array overload. For new UDTF to work, override the deprecated method was required which is not correct. ### Does this PR introduce _any_ user-facing change? - User-Facing Bug Fix: Users and 3rd-party developers can now write and execute GenericUDTFs implementing only initialize(StructObjectInspector) - Backward Compatibility: Existing built-in UDTFs (e.g., explode, json_tuple, stack) still override initialize(ObjectInspector[]) continue to work without any changes. ### How was this patch tested? Added a Junit test -- 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]
