Skip to content

Conversation

@pitrou
Copy link
Member

@pitrou pitrou commented Nov 12, 2025

Fuzz regression files for the IPC file reader should be stored in data/arrow-ipc-file, not data/arrow-ipc-stream.

Fortunately, those regression files still pass fine despite not having been tested.

Fuzz regression files for the IPC file reader should be stored in `data/arrow-ipc-file`, not `data/arrow-ipc-stream`.

Fortunately, those regression files still pass fine despite not having been tested.
@pitrou pitrou requested review from WillAyd and alamb November 12, 2025 10:27
Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I verified that all these files end with ARROW1 (aka are actually arrow files) 👍

Thank you @pitrou and @WillAyd

@pitrou pitrou merged commit 9a56b47 into apache:master Nov 12, 2025
@pitrou pitrou deleted the gh48107-fuzz-files-directory branch November 12, 2025 14:54
pitrou added a commit to pitrou/arrow that referenced this pull request Nov 12, 2025
Update `arrow/testing` submodule to latest git master after apache/arrow-testing#119
pitrou added a commit to apache/arrow that referenced this pull request Nov 13, 2025
### Rationale for this change

Update `arrow/testing` submodule to latest git master after apache/arrow-testing#119

### Are these changes tested?

By existing CI tests (particularly the ASAN/UBSAN build).

### Are there any user-facing changes?

No.
* GitHub Issue: #48107

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants