jackylee-ch commented on PR #825: URL: https://github.com/apache/paimon-rust/pull/825#issuecomment-5676152065
Closing this — #831 already covers it. That change walks the writer's own field list for the deletion-vector item record, which is what this PR did, and goes further: it accepts a plain (non-union) item as PyPaimon writes it, where this PR would have rejected the schema outright, and it tolerates reordered and unknown fields. Its `test_deletion_vectors_follow_writer_item_schema` already covers the two-entry, two-item shape across both nullable items and a missing `_CARDINALITY`. Your pushback was right: the version-compatibility framing was the weak part of this PR. The mechanism was the substance, and it has landed. -- 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]
