carols10cents commented on pull request #1887: URL: https://github.com/apache/arrow-datafusion/pull/1887#issuecomment-1061144961
@alamb @tustvold Ok, I think this is ready for re-review, all the `Any` stuff is gone :) > FYI the [datafusion-contrib/datafusion-substrait](https://github.com/datafusion-contrib/datafusion-substrait) repo from @andygrove may be related to this (as in maybe it eventually removes protobuf serialization). > > Perhaps to plan for that eventually we could keep the serialization API operating on an opaque format (like `fn serialize(expr: Expr) -> Vec<u8>`) 🤔 That should be pretty easy to add - would you like me to do that in this PR or in a future PR? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
