alamb commented on issue #4426: URL: https://github.com/apache/arrow-datafusion/issues/4426#issuecomment-1332613823
Thanks @NGA-TRAN -- like the proposal in general. I think the trickiest part will be "where to store the prepared statement" (aka `my_plan_2`) And I think it is critical for this feature to be opt in (so downstream crates can decide if/how they want to handle the state management) I suggest following the model of CreateExternalTable https://github.com/apache/arrow-datafusion/blob/23b4495516a5a6246e1732bca4f82bb065b021be/datafusion/expr/src/logical_plan/plan.rs#L1335-L1358 Where the core of DataFusion knows how to create the `LogicalPlan::CreateExternalTable` but leaves it to the downstream system (aka `datafusion-cli` or IOx) to actually execute them by connecting them together. @Dandandan and @andygrove any thoughts? -- 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]
