tanmayrauth commented on PR #1131:
URL: https://github.com/apache/iceberg-go/pull/1131#issuecomment-4570516634
@laskoviymishka I've made the below change PTAL when you have a moment.
- Dropped the two Go-writes-Go-reads tests — TestDVWriterSingleDataFile /
TestDVWriterMultipleDataFiles already cover that path.
- Added TestCrossClientReadJavaSingleBlobDV and
TestCrossClientReadJavaMultiBlobDV — both read Java-produced .puffin artifacts
end-to-end through ReadDV.
- Scoped down the byte-equality test to a single-array-container fixture
and renamed it accordingly; comment now calls out the Java RLE step as the
reason byte-match doesn't generalize.
- Added a high-key rejection test mirroring Java's
testDeserializeUnsupportedRoaringExample so both clients fail-closed on the
same spec-illegal input.
--
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]
