rambleraptor opened a new pull request, #2822:
URL: https://github.com/apache/iceberg-python/pull/2822
<!--
Thanks for opening a pull request!
-->
<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
Part of #2261
# Rationale for this change
This adds a PuffinWriter for writing deletion vectors.
Right now, it's just the writer class + some round trip tests (where we read
+ write the same file) to sanity check that the PuffinWriter works as expected.
Writing Puffin files is very complex, so I wanted to make sure we all agreed on
the writing semantics before using this elsewhere.
Let me know your thoughts on this (or if it's too granular)
## Are these changes tested?
Unit tests included
## Are there any user-facing changes?
<!-- In the case of user-facing changes, please add the changelog label. -->
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
