You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/12/23 20:24:29 UTC

[GitHub] [arrow-datafusion] houqp commented on issue #1480: Ballista: Partition columns are duplicated in protobuf decoding.

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org