wirybeaver commented on PR #22988: URL: https://github.com/apache/datafusion/pull/22988#issuecomment-5222696095
> Thanks @wirybeaver -- I am sorry for the delay > > I think some SLT tests will help thsi too (I will make a follow on PR) > > Can you file some follow on tickets for the following follow on items? > > 1. EXPLAIN shows nothing about the merge (e.g. no ON conditions, WHEN clause, etc) > 2. Update `docs/source/user-guide/sql/dml.md` to mention MERGE INTO. > > We may also want to consider implementing a basic MERGE INTO for MemTable (keep it as simple as possible to show the design) Thanks for the guidance. Will follow your suggestion to create a follow up PR. Moreover, I also wanted to address the two edge cases that I mentioned in the summary caused by the target alias replacement. -- 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]
