MarkCruz911 opened a new pull request, #54922: URL: https://github.com/apache/arrow/pull/54922
Rationale for this change Closes #49717. The from_buffers documentation should explain the caller's responsibility to provide valid Arrow data and the risk of crashes when using invalid arrays. What changes are included in this PR? Added warnings to the from_buffers docstrings for: Array StringArray LargeStringArray DictionaryArray The warnings explain that construction performs basic validation and recommend calling validate(full=True) on the resulting array before using it. RunEndEncodedArray.from_buffers was also reviewed. It delegates to _from_arrays, which already calls validate(full=True), so it was left unchanged. Are these changes tested? git diff --check passed. Reviewed the diff to confirm that only the four docstrings were changed. Inspected the constructor implementations and their validation calls. The PyArrow test suite and documentation build have not been run. Are there any user-facing changes? The Python API documentation now explains validation responsibilities and the risk of crashes from invalid data. There are no changes to runtime behavior or public API signatures. **PR code and description written by:** - [x] Human - [x] AI **Reviewed before submission by:** - [x] Human - [x] AI - [ ] Not reviewed -- 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]
