[GitHub] [arrow-datafusion] houqp opened a new issue #1962: Merge Dataframe trait and DataframeImpl into Dataframe struct

[GitBox <gi...@apache.org>]

houqp opened a new issue #1962:
URL: https://github.com/apache/arrow-datafusion/issues/1962


   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   
   Follow up for https://github.com/apache/arrow-datafusion/pull/1712#issuecomment-1025266914.
   
   The Dataframe trait was introduced in order to have two separate Dataframe implementation in Datafusion and Ballista. Since then, the design has changed and we are sharing the same dataframe implementation between Datafusion and Ballista now. Therefore, we could simplify the code base by getting rid of the Dataframe trait abstraction and promoting DataframeImpl into Dataframe struct.
   
   **Describe the solution you'd like**
   
   Dataframe trait and DataframeImpl merged into a single Dataframe struct
   
   
   **Additional context**
   
   Should help unblock https://github.com/apache/arrow-datafusion/pull/1712
   


-- 
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

[GitHub] [arrow-datafusion] yjshen closed issue #1962: Merge Dataframe trait and DataframeImpl into Dataframe struct

[GitBox <gi...@apache.org>]

yjshen closed issue #1962:
URL: https://github.com/apache/arrow-datafusion/issues/1962


   


-- 
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