Skip to content

Conversation

@timsaucer
Copy link
Member

@timsaucer timsaucer commented Oct 3, 2024

Which issue does this PR close?

Closes #869

Rationale for this change

User requested, and also opens up functionality in datafusion-ray

What changes are included in this PR?

Exposes to_proto and from_proto on logical and physical plans.

Are there any user-facing changes?

Pure addition.

TODO:

  • Add unit tests
  • Add python wrappers

Copy link
Member

@andygrove andygrove left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome. Thanks @timsaucer

@timsaucer timsaucer marked this pull request as ready for review October 4, 2024 14:59
@timsaucer timsaucer mentioned this pull request Oct 4, 2024
@timsaucer timsaucer merged commit d181a30 into apache:main Oct 4, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bindings to datafusion-proto

2 participants