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/09 15:06:38 UTC

[GitHub] [arrow] andygrove commented on pull request #8870: ARROW-10854: [Rust] [DataFusion] Simplify logical plan scans

andygrove commented on pull request #8870:
URL: https://github.com/apache/arrow/pull/8870#issuecomment-741830810


   This is potentially going to be a breaking change for my use of the DataFusion logical plan in Ballista because I need to serialize logical plans to protobuf format to transmit them over the network. With the explicit variants for each type of input I have access to all the information I need, but with a reference to a `dyn TableProvider` this becomes opaque. I should have added tests for this use case.
   
   I will have a think about how I can work around this as well. Maybe there are other approaches.


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