ad1happy2go commented on PR #19005: URL: https://github.com/apache/hudi/pull/19005#issuecomment-4893049838
@danny0405 @yihua — does the explanation above make sense? To recap the core argument: the table type (COW vs MOR) is a storage detail and shouldn't change the result of an incremental query, and the merge here isn't unbounded — it's scoped to the incremental window's commit-time range via `InstantRange`, exactly the window COW already uses. So MOR ends up returning the same fully-materialized rows COW does for that window. If that reasoning holds, could you help review this PR? Happy to walk through any specific scenario where the expected output still seems ambiguous. -- 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]
