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/03/02 05:12:34 UTC

[GitHub] [arrow-datafusion] matthewmturner commented on pull request #1893: Add write_ipc to ExecutionContext

matthewmturner commented on pull request #1893:
URL: https://github.com/apache/arrow-datafusion/pull/1893#issuecomment-1056242541


   @alamb sry i had basically copied this from `write_parquet` which was source of both your comments.
   
   I dont think the clone on the plan was needed, i removed it in `write_ipc`, `write_parquet`, and `write_csv`.  If you would rather i do those other functions in a separate PR i can do that.
   
   I think `IpcWriteOptions` should be clone - so im going to submit PR for that in arrow.


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