Use `std::num::NonZero` for args that must be greater than 0
Open
api-change
next-major-release
- Dominant language
- Rust
- Stars
- 3.6k
- Forks
- 1.3k
- Avg merge
- 2d 18h
- Merged PRs (30d)
- 169
Description
Maybe at some point we could change this to use `NonZeroUsize`?
_Originally posted by @mbrobbel in https://github.com/apache/arrow-rs/pull/7558#discussion_r2113252581_
Contributor guide
Research direction
No file, test, or entry point is named in the issue. Start by locating Rust APIs in arrow-rs that accept arguments required to be greater than zero, then review their callers and tests; done means the affected scope and the NonZeroUsize migration are clearly established.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- backend-api-design
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100