alamb commented on PR #5322: URL: https://github.com/apache/arrow-datafusion/pull/5322#issuecomment-1442621079
> This sounds good to me too. We will need to do this at some point, physical plan code is getting quite big. I believe the issue was that that last time we tried a bunch of SessionContext and the traits it used (like TableProvider) were all needed so there was no clean way to split out the physical plans Now that @tustvold made a harder split between SessionState and SessionContext it may be more tractable now. I'll plan to file / record the follow on items as tickets prior to merging this PR -- 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...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
