alamb opened a new pull request, #4779: URL: https://github.com/apache/arrow-datafusion/pull/4779
# Which issue does this PR close? Closes https://github.com/apache/arrow-datafusion/issues/4065 from @andygrove Closes https://github.com/apache/arrow-datafusion/issues/4728 from @Jefffrey Also reported by @maxburke in slack # Rationale for this change Debug builds are overflowing the stack causing pain and suffering for users. # What changes are included in this PR? 1. Extract several parts of the body of `SqlToRel::sql_expr_to_logical_expr` into separate functions The reason this reduces stack size in debug builds is explained in the "Technical Backstory" heading of https://github.com/apache/arrow-datafusion/pull/1047 # Are these changes tested? Covered by existing coverage # Are there any user-facing changes? None intended (other than avoiding stack overflows) -- 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]
