[GitHub] [arrow-datafusion] avantgardnerio commented on issue #5076: Re-implement `DataFrame.write_*` to use `LogicalPlan::Write`

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

avantgardnerio commented on issue #5076:
URL: https://github.com/apache/arrow-datafusion/issues/5076#issuecomment-1523500363

   I don't have a strong opinion here, since I haven't had much time to look in depth and we fork execution at the LogicalPlan level before it ever gets to the physical plan anyway. 
   
   I do see how adding support for DML in TableProvider and PhysicalExec would be the next logical step. I would like to weigh in and say if we do that:
   
   1. Don't forget there are folks like me that do have the ability to mutate data. I have no idea how this would work in the broader `arrow-rs` sense though (happy to talk about how we do it)
   2. If it is for write-once type things like `CTAS` we probably want some `TableProviderFactory` support for it rather than `TableProvider`


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