wirybeaver commented on PR #24195: URL: https://github.com/apache/datafusion/pull/24195#issuecomment-5324188353
@timsaucer Thanks for catching this. Fixed in e2e364723: `MemTable` now returns a deferred `MergeIntoExec`, so source collection and table mutation happen only when the physical plan is executed. Plain `EXPLAIN MERGE INTO` now builds and displays the plan without mutating the target; `EXPLAIN ANALYZE` still executes it as expected. I also added a regression test that verifies the target rows remain unchanged after `EXPLAIN MERGE INTO`. The pre-existing eager UPDATE/DELETE behavior is separate from this PR. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
