[PR] fix(parquet/encoding): reject invalid delta byte prefixes [arrow-go]

["fallintoplace (via GitHub)" <gi...@apache.org>]

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] fix(parquet/encoding): reject invalid delta byte prefixes [arrow-go]

["zeroshade (via GitHub)" <gi...@apache.org>]

zeroshade merged PR #1014:
URL: https://github.com/apache/arrow-go/pull/1014


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org