JingsongLi commented on PR #9899: URL: https://github.com/apache/paimon/pull/9899#issuecomment-5708353412
Right fix. The old fallback `group.get(0)` is not guaranteed to be a full-range normal file — with a blob or vector-store file first in the group, the split built from it covers only a sub-range and the read would emit fewer rows than the group contains. Choosing the anchor (which skips blob/vector files and picks the min-sequence full-range normal file) makes the representative correct by construction, and the extracted static overload makes this unit-testable, which the new test uses well. Small nit: `retrieveAnchorFile`'s `checkState` message still says "Data-evolution deletion vectors should have a normal anchor file..." while it is now also reached with deletion vectors disabled; consider rewording it to fit both callers. -- 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]
