fallintoplace opened a new pull request, #1161: URL: https://github.com/apache/arrow-go/pull/1161
### Rationale for this change When decimals are stored as Parquet integers, values outside the declared precision can be narrowed to int32 or int64 without a runtime error. That can silently write a different value. ### What changes are included in this PR? Validate non-null Decimal128 and Decimal256 values against their declared precision before integer conversion, and return arrow.ErrInvalid when a value does not fit. ### Are these changes tested? Yes. The test covers Decimal128 and Decimal256 with both int32 and int64 Parquet storage. The full parquet/pqarrow package suite passes. ### Are there any user-facing changes? Invalid decimal arrays now return an error instead of producing corrupted integer values. -- 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]
