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 2022/02/01 00:19:15 UTC

[GitHub] [arrow-datafusion] realno commented on pull request #1712: generalize table provider df impl

realno commented on pull request #1712:
URL: https://github.com/apache/arrow-datafusion/pull/1712#issuecomment-1026343875


   > do you think it makes sense to just replace DataFrameImpl with Dataframe now and get rid of all the trait object overheads?
   
   I like the idea of simplifying interfaces - given the issues with this PR I think it seems to be a good idea to just consolidate `DataFrame`. Please take this as an opinion given my limited knowledge of the code base. Another thought is, will this have any impact on things like Python library? 


-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org