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 2022/11/23 11:46:16 UTC
[GitHub] [arrow-datafusion] alamb opened a new issue, #4338: Add serialization for entire LogicalPlans to datafusion-proto
alamb opened a new issue, #4338:
URL: https://github.com/apache/arrow-datafusion/issues/4338
**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
In order to serialize subquery expressions (https://github.com/apache/arrow-datafusion/issues/4337) we would need to serialize entire `LogicalPlans` which `datafusion-proto` does not do .
**Describe the solution you'd like**
I would like to have the ability to serialize / deserialize `LogicalPlans` in the datafusion. I believe we can take a friendly look / downright copy this code from ballista:
https://github.com/apache/arrow-ballista/blob/acbdf608807697e72122a249c4bf7505274906d0/ballista/core/proto/datafusion.proto
**Describe alternatives you've considered**
A clear and concise description of any alternative solutions or features you've considered.
**Additional context**
https://github.com/apache/arrow-datafusion/issues/4337
cc @andygrove @thinkharderdev @avantgardnerio
I am marking this as "good first isseu" because I think it would mostly be an exercise in porting code and tests from one repo to another.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-datafusion] l0kr commented on issue #4338: Add serialization for entire LogicalPlans to datafusion-proto
Posted by "l0kr (via GitHub)" <gi...@apache.org>.
l0kr commented on issue #4338:
URL: https://github.com/apache/arrow-datafusion/issues/4338#issuecomment-1462938156
Let me take a stab at this. @alamb anything changed since this task was created?
--
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
[GitHub] [arrow-datafusion] alamb commented on issue #4338: Add serialization for entire LogicalPlans to datafusion-proto
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb commented on issue #4338:
URL: https://github.com/apache/arrow-datafusion/issues/4338#issuecomment-1463756648
THanks @l0kr -- sorry for the old ticket. I think this task is actually is mostly done https://github.com/apache/arrow-datafusion/tree/main/datafusion/proto/src/logical_plan
It would be very valuable if you were able to go through and fill out the various "Not implemented" in the plan conversion.
Closing
--
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
[GitHub] [arrow-datafusion] alamb closed issue #4338: Add serialization for entire LogicalPlans to datafusion-proto
Posted by "alamb (via GitHub)" <gi...@apache.org>.
alamb closed issue #4338: Add serialization for entire LogicalPlans to datafusion-proto
URL: https://github.com/apache/arrow-datafusion/issues/4338
--
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