alamb opened a new pull request, #1546: URL: https://github.com/apache/arrow-rs/pull/1546
# Which issue does this PR close? Closes https://github.com/apache/arrow-rs/issues/1544 # Rationale for this change We have a full validation mode that validates the full contents of creating Arrays but it is (on purpose) not called in many places in arrow for performance reasons: https://docs.rs/arrow/11.1.0/arrow/array/struct.ArrayData.html#method.validate_full This leads to two possible issues: 1. Lack of test coverage of arrays constructed within arrow 2. Possible Lack of test coverage of the validation routine itself # What changes are included in this PR? 1. Add a `force_validate` feature flag that forced the validation check for *all* array creations (defaults to off) 2. Add a new CI check that runs with this flag on # Are there any user-facing changes? New optional feature `force_validate` -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
