Omega359 commented on issue #13444: URL: https://github.com/apache/datafusion/issues/13444#issuecomment-2480119872
I think this may have been a pre-existing issue. The core issue I think is that apparently md5 is required to return utf8: https://github.com/apache/datafusion/blob/6d8313ebc865f9bff007bfc04652f58b016cbc1b/datafusion/functions/src/crypto/basic.rs#L190 but the return_type function says it can return other types: ```Rust fn return_type(&self, arg_types: &[DataType]) -> Result<DataType> { use DataType::*; Ok(match &arg_types[0] { LargeUtf8 | LargeBinary => LargeUtf8, Utf8View | Utf8 | Binary => Utf8, Null => Null, Dictionary(_, t) => match **t { LargeUtf8 | LargeBinary => LargeUtf8, Utf8 | Binary => Utf8, Null => Null, _ => { return plan_err!( "the md5 can only accept strings but got {:?}", **t ); } }, other => { return plan_err!( "The md5 function can only accept strings. Got {other}" ); } }) } ``` Should be an easy fix to the return_type fn. -- 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]
