ZZZxDong commented on PR #8837: URL: https://github.com/apache/paimon/pull/8837#issuecomment-5089257170
The remaining CI failure surfaced a real semantic difference: -D rows from the fast path carry NULL for non-key columns, so incremental / audit_log reads would not see the old field values of deleted rows (the TVF test asserts `[-D, 1, 11]` but got `[-D, 1, null]`). Since this affects the changelog contract for any deduplicate table, I made the fast path opt-in: it is now off by default and enabled via `spark.paimon.delete.point-delete.enabled`, with the trade-off documented on the option. Default behavior is completely unchanged. Happy to adjust if you prefer a different approach. -- 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]
