fallintoplace opened a new pull request, #954: URL: https://github.com/apache/arrow-go/pull/954
### Rationale for this change Malformed compressed page data should be reported as a Parquet read error, but the existing block codec API has no error return and built-in codecs panic on decode failures. Refs #949 ### What changes are included in this PR? - Add an optional ErrorCodec interface and a safe Decode adapter without breaking existing registered codecs. - Implement error-returning block decoding for Snappy, Gzip, Brotli, Zstd, LZ4_RAW, and the uncompressed codec. - Recover decode panics from legacy custom codecs in one compatibility adapter. - Route Parquet page decompression through the adapter and add contextual errors. - Validate complete gzip and Brotli output, including checksum and trailing output when a destination is supplied. ### Are these changes tested? Yes. Tests cover malformed input for every built-in compressed block codec and panic recovery for a legacy codec. The compression package passes its race test, and focused page-reader tests pass. ### Are there any user-facing changes? The API change is additive. Existing Codec implementations continue to compile, while codecs can implement ErrorCodec to return decoding failures directly. File reads now return malformed-compression errors instead of exposing codec panics. -- 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]
