alamb commented on code in PR #14631:
URL: https://github.com/apache/datafusion/pull/14631#discussion_r1954394569


##########
datafusion/core/src/datasource/memory.rs:
##########
@@ -648,9 +649,14 @@ mod tests {
         // Create a table scan logical plan to read from the source table
         let scan_plan = LogicalPlanBuilder::scan("source", source, 
None)?.build()?;
         // Create an insert plan to insert the source data into the initial 
table
-        let insert_into_table =
-            LogicalPlanBuilder::insert_into(scan_plan, "t", &schema, 
InsertOp::Append)?
-                .build()?;
+        let insert_into_table = LogicalPlanBuilder::insert_into(
+            scan_plan,
+            "t",
+            table_sink,
+            &schema,

Review Comment:
   if it is the scan schema, then it should be possible to get from the 
`scan_plan` 🤔 



-- 
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-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to