linliu-code commented on PR #19934: URL: https://github.com/apache/hudi/pull/19934#issuecomment-5704671209
Closing unmerged — we are reversing the direction of this fix. Instead of making COW `MERGE INTO ... WHEN MATCHED THEN DELETE` honor event time ordering, the plan is now to make **MOR ignore** the ordering value for SQL merge-into deletes, so both table types apply a SQL-issued delete unconditionally. Rationale: `DELETE FROM` already ignores ordering values on both table types and both merge modes (see `TestDeleteFromTable`), so aligning MIT with it is more consistent with the intent of a SQL delete than making MIT the one SQL surface where a delete can be silently dropped. Ingestion paths (Deltastreamer / CDC) keep respecting ordering values and are unaffected. Leaving #17372 open: the inconsistency it reports is real, only the intended resolution direction has changed. A replacement PR will follow. -- 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]
