alamb commented on issue #22393: URL: https://github.com/apache/datafusion/issues/22393#issuecomment-5297037354
> Ok, [#24377](https://github.com/apache/datafusion/pull/24377) is ready for the backport of [#24365](https://github.com/apache/datafusion/pull/24365) into `branch-55`. I approved and merge this > I'm looking at [#24195](https://github.com/apache/datafusion/pull/24195) which looks like it's probably important because without it we're releasing the `MERGE INTO` without implementation for our memory table. 🤔 It has a merge conflict [@wirybeaver](https://github.com/wirybeaver) but I'll start reviewing it. I think this is something we could do as a follow on (e.g. 55.1 or just in 56) as it is a net new feature The rationale for including MERGE INTO support as I understand it, is to allow downstream consumers such as Iceberg to implement merge (they have their own TableProvider so they won't use the MemoryTable implementation anyways for testing -- 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]
