wgtmac commented on PR #172: URL: https://github.com/apache/iceberg-cpp/pull/172#issuecomment-3186296602
More test cases will be added progressively because we only have a basic reader implementation: - We are adding test manifest list and manifest files to verify manifest reader implementations are basically correct: https://github.com/apache/iceberg-cpp/pull/168 and https://github.com/apache/iceberg-cpp/pull/171. But this heavily depends on the generated files and not easy to produce various files. - We are on the way to implement manifest writers so we can add round trip test to verify v1/v2/v3 compatibility. This can help test all the required/optional fields like this PR. - We will add spark docker integration test once we can run e2e table scan. -- 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]
