apache / apache/arrow-nanoarrow
Skip pre-1.0.0 IPC integration tests
- 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