chenjian2664 opened a new pull request, #4311:
URL: https://github.com/apache/iceberg-python/pull/4311
<!--
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. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->
# Rationale for this change
No unit test reads or writes a snappy or bzip2 Avro block. The snappy
checksum bug fixed in #2252 made every snappy-compressed Avro file unreadable,
and nothing caught it. Exchanging files with fastavro in both directions checks
each codec's block framing against an independent implementation of the Avro
spec. A test that only compresses and decompresses with PyIceberg would miss a
framing error that both sides share.
## Are these changes tested?
## 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]
