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: [email protected]
