jizezhang commented on issue #16944: URL: https://github.com/apache/datafusion/issues/16944#issuecomment-3508846189
Hi, if no one is actively working on this task, I am interested. To check my understanding, it seems like the goal is to move some functionality from ballista codec to `datafusion_proto::logical_plan::DefaultLogicalExtensionCodec`? Specifically, - add a `default` method for `DefaultLogicalExtensionCodec` that populates a list of file format codecs under `datafusion_proto::logical_plan::file_formats` module - add a `FileFormatProto` struct as in ballista - implement `try_decode_file_format` and `try_encode_file_format` based on ballista implementation? If those sound correct, I can work on getting a PR. Thanks! -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
