Omega359 commented on PR #13404: URL: https://github.com/apache/datafusion/pull/13404#issuecomment-2501120749
I don't agree with the decision to not support implicit coercion to string for function arguments. Firstly, it's a regression from currently supported features and will break things for users, myself included. Secondly Postgresql's requirement for explicit casting for so many things imho is one of it's most annoying 'features' and is generally not required by just about every other database out there. The reasoning given for the DuckDB decision was nothing to do with functions and everything to do with casting issues with general sql and comparisons which I tend to agree with. See the [PR for details](https://github.com/duckdb/duckdb/pull/10115). `select length(42:VARCHAR);` should not be required. `select length(42);` should just work. -- 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
