Jefffrey commented on code in PR #5520:
URL: https://github.com/apache/arrow-datafusion/pull/5520#discussion_r1132012053
##########
datafusion/expr/src/logical_plan/builder.rs:
##########
@@ -203,6 +203,24 @@ impl LogicalPlanBuilder {
Self::scan_with_filters(table_name, table_source, projection, vec![])
}
+ /// Convert a logical plan into a builder with a [DmlStatement]
+ pub fn insert_into(
+ input: LogicalPlan,
+ table_name: impl Into<String>,
Review Comment:
I think it would just involve changing from:
```rust
table_name: impl Into<String>
```
to:
```rust
table_name: impl Into<OwnedTableReference>
```
and use that directly, if #5343 gets merged first. Since there will be a
`From<String>` implementation for `OwnedTableReference` anyway
--
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]
