[PR] Minor: split EXPLAIN and ANALYZE planning into different functions [datafusion]
alamb opened a new pull request, #15188: URL: https://github.com/apache/datafusion/pull/15188 ## Which issue does this PR close? - part of https://github.com/apache/datafusion/issues/15021 - Part of https://github.com/apache/datafusion/pull/15166 ## Rationale for this change I would like to make it easier to review https://github.com/apache/datafusion/pull/15166 so I pulled out a mechanical change into a new PR ## What changes are included in this PR? 1. Break the handling of `explain` and `analyze` into two functions (with much less indent!) ## Are these changes tested? By existing tests ## Are there any user-facing changes? -- 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
Re: [PR] Minor: split EXPLAIN and ANALYZE planning into different functions [datafusion]
alamb merged PR #15188: URL: https://github.com/apache/datafusion/pull/15188 -- 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
Re: [PR] Minor: split EXPLAIN and ANALYZE planning into different functions [datafusion]
alamb commented on PR #15188: URL: https://github.com/apache/datafusion/pull/15188#issuecomment-2718829842 Thanks for the review @waynexia ! Since this is a small refactoring I will merge this one in so I can get the real PR ready for review! -- 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
