jorgecarleitao commented on pull request #8870: URL: https://github.com/apache/arrow/pull/8870#issuecomment-741845545
@andygrove that is a good point that I had not thought about. Unfortunately, I have also not though about them when reviewing the addition of `LogicalPlan::Extension`, `Expr::ScalarUDF`, and `Expr::AggregateUDF`, all of which directly or indirectly use `Arc<dyn X>`. Is this already a concern for Ballista? Would it be possible to require `TableProvider` to implement [`serde::ser::Serialize`](https://docs.serde.rs/serde/ser/trait.Serialize.html), and use something like https://github.com/dtolnay/erased-serde to serialize trait objects? Also relevant: https://stackoverflow.com/a/50026869/931303 ---------------------------------------------------------------- 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]
