apache / apache/arrow-nanoarrow

Skip pre-1.0.0 IPC integration tests

Open
#648 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C
Stars
249
Forks
68
Avg merge
2d 21h
Merged PRs (30d)
11

Description

In the process of actually supporting pre 1.0.0 messages for the integration testing (#643), I figured out where we can just skip those tests:

https://github.com/apache/arrow/blob/7cc07681189e71ef986653129a6804a81d32ce8f/dev/archery/archery/integration/runner.py#L179

I don't think there is any need to support pre-1.0.0 IPC streams in the long term, although I think it is good to support them initially to ensure we are starting on the same level as all the other Arrow implementations.

Contributor guide

No contributing guide indexed for this repository

Research direction

Open dev/archery/archery/integration/runner.py around line 179 and inspect how IPC integration tests are selected. Confirm which pre-1.0.0 messages or streams should be skipped, then verify that the integration test runner no longer attempts those cases while supported messages remain covered.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
testing
Issue type
Refactor
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.