aharpervc commented on code in PR #1808: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1808#discussion_r2045317219
########## src/parser/mod.rs: ########## @@ -15017,6 +15075,13 @@ impl<'a> Parser<'a> { } } + fn parse_return(&mut self) -> Result<Statement, ParserError> { + let expr = self.parse_expr()?; Review Comment: > Maybe we can remove the return statement bits from this PR and tackle it separately as its own PR? I don't think this is possible, since SQL Server requires that functions have a return, so create function goes together with return. I'll take another pass on the PR though -- 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