MINOR: [Docs] Mention validate() helpers in Python IPC docs#49254
MINOR: [Docs] Mention validate() helpers in Python IPC docs#49254shashbha14 wants to merge 1 commit intoapache:mainfrom
Conversation
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or See also: |
raulcd
left a comment
There was a problem hiding this comment.
I am unsure this really tackles the issue that was raised. The issue has two parts one about the new security guidelines:
https://github.com/apache/arrow/blob/68d13685b4e3914ff2b36eeca606ae428b8e3485/docs/source/format/Security.rst
and two a question on whether a new flag on the IPCReadOptions like always_validate makes sense or not.
This PR is not tackling any of those concerns.
|
Thanks for the review and clarification.You’re right, this PR doesn’t address the Security.rst text or the |
This PR adds a short “Security considerations for untrusted IPC data” section
to the Python IPC docs. It:
reading IPC data from untrusted or semi‑trusted sources.
RecordBatch.validate()andTable.validate()as the relevantvalidation helpers.
pa.ipc.open_streamand
pa.ipc.open_file.It doesn’t try to answer the broader questions in #49241 (updates to
format/Security.rst or adding an
always_validateoption), but it shouldstill make the validation APIs more discoverable for PyArrow users.