You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/12/14 14:04:55 UTC

[GitHub] [arrow] jorgecarleitao commented on pull request #8910: ARROW-10783: [Rust] [DataFusion] Resolve TableScan provider eagerly

jorgecarleitao commented on pull request #8910:
URL: https://github.com/apache/arrow/pull/8910#issuecomment-744461751


   Funny thing, I was planning to work on this exact same change ❤️ (removing the `TableProvider`, offering the provider directly, and making the planner call `provider.scan`). I agree with this design  ^_^
   
   Rename also looks good to me.


----------------------------------------------------------------
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:
users@infra.apache.org