comphead commented on code in PR #13925: URL: https://github.com/apache/datafusion/pull/13925#discussion_r1899698871
########## docs/source/user-guide/sql/scalar_functions.md: ########## @@ -971,8 +971,8 @@ concat_ws(separator, str[, ..., str_n]) #### Arguments -- **separator**: Separator to insert between concatenated strings. - **str**: String expression to operate on. Can be a constant, column, or function, and any combination of operators. Review Comment: I think `separator` should come first in order like it is in the function signature -- 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
