alamb commented on code in PR #6313:
URL: https://github.com/apache/arrow-datafusion/pull/6313#discussion_r1191157115


##########
datafusion/core/src/execution/context.rs:
##########
@@ -450,6 +457,36 @@ impl SessionContext {
         self.read_batch(record_batch)
     }
 
+    // Execute a COPY TO statement, returning the number of rows

Review Comment:
   Thanks @ozankabak  -- I have reviewed 
https://github.com/synnada-ai/arrow-datafusion/pull/89 and I have thought about 
how to incorporate the same structure
   
   I really like your idea to use the the same plans for `COPY TO` and `INSERT 
INTO`. After some more thought, I have an idea of how to plan `COPY` statements 
using the same plans as an INSERT. 
   
   Here is a proposal for a simplified API: 
https://github.com/apache/arrow-datafusion/issues/6339
   
   I'll try and bang out a Proposal shortly



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