Describe the bug

I am trying to upgrade the version of DataFusion that Comet uses. Comet currently relies on ScalarUDFImpl.return_type to get the return type of a scalar UDF. This now returns an internal error when called for DatePartFunc.

    fn return_type(&self, _arg_types: &[DataType]) -> Result<DataType> {
        internal_err!("return_type_from_exprs shoud be called instead")
    }

Should we deprecate this function if it is no longer the correct approach for determining UDF return types? It looks like return_type_from_exprs should now be used instead?

To Reproduce

No response

Expected behavior

No response

Additional context

No response