test: add cross-language deletion vector fixtures - #913
wirybeaver wants to merge 1 commit into
Conversation
b5c2630 to
dd3384a
Compare
|
Thanks for working on these! Let me know when you think they are ready for review. |
|
@wgtmac It's my pleasure. Will have a self-review and open PRs by end of this weekend! |
|
I don't think this should block other DV PRs and I think basic DV tests should be self-contained. For cross-language compatibility, we may make use of the newly created iceberg-verification project. |
|
@manuzhang Yeah, I also reconsidering to refactor the PR order of #916 to focus on delivering features first and conflate the validation PRs into the feature PRs. Thanks for pointing out the iceberg-verification project. |
|
Closing the standalone fixture PR as part of the four-feature consolidation in #916. Focused tests now accompany the delivered features; this fixture-only prerequisite is no longer part of the roadmap. |
Summary
Partially close #916
This adds cross-language coverage beyond C++ self-roundtrips and follows up on the deletion-vector work discussed in #637 and #777.
Testing
puffin_test(31/31 tests passed with GCC 14)3020adbbc3faff047da6f483f739f1b5e1de611band compared them byte-for-byte