comphead commented on PR #14511: URL: https://github.com/apache/datafusion/pull/14511#issuecomment-2637374150
> > I would prefer the function name to be highlighted some how, it can uppercase or square brackets, otherwise the function name got lost in the message > > 1. `Function test expects NativeType::Numeric but received NativeType::String` > 2. `Function [test] expects NativeType::Numeric but received NativeType::String` > 3. `Function TEST expects NativeType::Numeric but received NativeType::String` > 4. `Function 'TEST' expects NativeType::Numeric but received NativeType::String` > 5. `Function 'test' expects NativeType::Numeric but received NativeType::String` > > my preference is the last version. Yeah, love it. Function name is not allowed to include special characters so it should be good -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
