etseidl opened a new pull request, #11981:
URL: https://github.com/apache/arrow-rs/pull/11981

   
   # Which issue does this PR close?
   
   - Closes #11769
   
   # Rationale for this change
   
   `PackedDecoder` can only be used when `max_def_level == 1`, but this 
invariant is not strictly enforced.
   
   # What changes are included in this PR?
   
   Change `PackedDecoder::new` signature to take `max_def_level` and panic if 
it is not `1`. A panic is chosen over returning an error because the choice to 
use `PackedDecoder` is not left to user input...trying to create one with any 
other `max_def_level` is a programming error.
   
   # Are these changes tested?
   
   Yes, new tests are added.
   
   # Are there any significant user-facing changes?
   
   No changes should be visible as the only calling site already ensures 
correct behavior.
   
   # AI disclosure 
   
   The fix and tests were created with the aid of an AI agent. 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to