milenkovicm opened a new pull request, #14631: URL: https://github.com/apache/datafusion/pull/14631
## Which issue does this PR close? this is more request for comment change, at the moment No PR at the moment, would open it after discussion - Closes #. ## Rationale for this change As discussed in https://github.com/apache/datafusion-ballista/issues/1164 DML statement has table reference instead of table source, which as implication impacts usability in case of ser/de logical plan. After deserialisation another table lookup need to be performed which impacts usability in case of ballista (where we keep two separate session context and only one of them has table definition) This PR add table source as well to LogicalPlan:DML to improve usability. ## What changes are included in this PR? this PR changes: - LogicalPlan:DML - proto LogicalPlanNode::DML ## Are these changes tested? using existent tests ## Are there any user-facing changes? proto definition changed -- 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
