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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to