houqp commented on issue #1480: URL: https://github.com/apache/arrow-datafusion/issues/1480#issuecomment-1000509287
Thanks @thinkharderdev for the report, this is due to: https://github.com/apache/arrow-datafusion/blob/68db579181bd826e6ab6cd659f52d443b950eaa5/datafusion/src/datasource/listing/table.rs#L128-L135 The fix should be pretty straight forward, we need a new constructor method for listing table for plan ser/de, which will just take the filed arguments and store them in the new struct as is without extra logic. Then we just need to call that new constructor method in https://github.com/apache/arrow-datafusion/blob/68db579181bd826e6ab6cd659f52d443b950eaa5/ballista/rust/core/src/serde/logical_plan/from_proto.rs#L200. -- 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]
