thinkharderdev commented on issue #1480:
URL: 
https://github.com/apache/arrow-datafusion/issues/1480#issuecomment-1000528396


   > 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
   > 
   > .
   
   Cool. I can take a crack at that. 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]


Reply via email to