Ensure `PackedDecoder` is not created incorrectly
Maintainers usually reply within 1 day
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 68/100
Research direction
Start by reading PackedDecoder and DefinitionLevelBufferDecoder::new in the arrow reader, then check how definition levels are validated nearby. Add checks that enforce max_def_level == 1 at the constructor and construction site; done means invalid values are rejected and the invariant is covered by relevant tests.
Written by the indexing model from the issue text.
Description
Is your feature request related to a problem or challenge?
Follow up to #11700. PackedDecoder in the arrow reader requires max_def_level == 1, but this is not enforced in either its constructor nor where constructed (DefinitionLevelBufferDecoder::new).
Describe the solution you'd like
Extra checks should be added to enforce the invariant.
Describe alternatives you've considered
No response
Additional context
No response
- Dominant language
- Rust
- Stars
- 3.6k
- Forks
- 1.4k
- Avg merge
- 2d 10h
- Merged PRs (30d)
- 326
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from apache/arrow-rs
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
Maintainers usually reply within 1 day
-
ipc: slicing buffer should not panic on invalid slicePossibly taken @wleeaf claimed this 3 days ago. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
apache/arrow-rs#11857 · 1 comment ·
Maintainers usually reply within 1 day
-
ipc: slicing union buffer should validate lengthPossibly taken @wleeaf claimed this 3 days ago. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
parquet: panic (index out of bounds) reading FIXED_LEN_BYTE_ARRAY dictionary page with out-of-range indexPossibly taken @efegokdemir claimed this 8 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 83/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 Under an hour Newbie friendliness 75/100
Maintainers usually reply within 1 day
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
Maintainers usually reply within 1 day
-
status:untriaged
Difficulty 1/5 Under an hour Newbie friendliness 72/100
midnightntwrk/midnight-zk#592 ·
Maintainers usually reply within 3 days
-
QuartoHub: have the Document name before the filename in the browser tabPossibly taken @cscheid claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
quarto-dev/q2#930 · 1 comment ·
Maintainers usually reply within 1 day
-
muse P3
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
HerringtonDarkholme/hd-lang#923 · 1 comment ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day