fallintoplace opened a new pull request, #1014:
URL: https://github.com/apache/arrow-go/pull/1014

   ### Rationale for this change
   
   DELTA_BYTE_ARRAY prefix lengths are used directly as slice bounds. Negative 
prefixes or prefixes longer than the previous value therefore panic. The 
decoder also retains its previous value when reused for a new page, even though 
each page starts an independent delta sequence.
   
   ### What changes are included in this PR?
   
   - Reset the previous value in `SetData`.
   - Propagate prefix stream decode errors and short reads.
   - Require the first prefix to be zero and validate later prefixes before 
slicing.
   - Apply the same validation to decode and discard paths.
   
   ### Are these changes tested?
   
   Yes. Focused tests cover decoder reuse, negative prefixes, and prefixes 
longer than the previous value.


-- 
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