returnString commented on pull request #8917: URL: https://github.com/apache/arrow/pull/8917#issuecomment-745164026
> FYI this will conflict with parts of #8910. IMO we should merge #8910 first, as it simplifies some of the code, making this PR easier. Yeah definitely; like I mentioned, that's actually _required_ for this to work with ordinary query methods (not just `read_table(provider)` as the tests are using currently) because we need a reference to the provider whilst doing logical plan optimisation :) ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
