leaves12138 commented on PR #9177: URL: https://github.com/apache/paimon/pull/9177#issuecomment-5254949684
Rechecked the latest update at `129068954c`. The previously reported correctness blocker is resolved: - normal-range connectivity is now derived exclusively from normal files, so a spanning BLOB/vector file cannot reconnect candidate ranges across an omitted normal range; - dedicated files are associated only after the normal contiguous groups are established; - `DataEvolutionNormalCompactTask` now fails fast on disjoint row-id inputs; - bounded manifests are pruned per candidate batch, while legacy manifests without row-id bounds consolidate candidates into one full scan. I reran the affected 174 core tests successfully. I also reran the 20,000-iteration normal and BLOB differential checks against full planning; both passed with the fix. The original spanning-BLOB reproduction now produces two independent normal tasks and no row-id-shifting task. I did not find another correctness issue in the updated implementation. The removal of `rewrite-row-ids=true` remains an explicit compatibility/product decision, but the documentation and exception no longer suggest a nonexistent replacement operation. -- 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]
