[GitHub] [arrow-datafusion-python] kylebrooks-8451 opened a new pull request, #344: Add interface to directly serialize Substrait plans to Python Bytes.

["kylebrooks-8451 (via GitHub)" <gi...@apache.org>]

kylebrooks-8451 opened a new pull request, #344:
URL: https://github.com/apache/arrow-datafusion-python/pull/344

   Also, Clean up existing Substrait bindings to return bytes instead of List[int].
   
   # Which issue does this PR close?
   
   Closes #343 
   
    # Rationale for this change
   What we had before didn't allow for serializing a Substrait plan to ProtoBuf. You had to use a SQL query which was awkward. This also fixes the `substrait` module return types to be `bytes` instead of `List[int]` which is also awkward.
   
   # What changes are included in this PR?
   Add `encode` method to `datafusion.substrait.plan`. Also change datafusion.substrait return types to be `bytes` instead of `List[int]`.
   
   # Are there any user-facing changes?
   Yes, but IMO changes for the better.
   


-- 
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-python] andygrove merged pull request #344: Add interface to serialize Substrait plans to Python Bytes.

["andygrove (via GitHub)" <gi...@apache.org>]

andygrove merged PR #344:
URL: https://github.com/apache/arrow-datafusion-python/pull/344


-- 
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