divjotarora opened a new pull request, #53522:
URL: https://github.com/apache/arrow/pull/53522

   ### Rationale for this change
   See https://github.com/apache/arrow/issues/53518 for rationale. Treating 
unknown compression codecs as `UNCOMPRESSED` is wrong and at best causes the 
read to fail late during decoding and at worst yields incorrect results if the 
compressed stream is actually a valid decodable set of bytes (very unlikely).
   
   ### What changes are included in this PR?
   This PR causes unsupported compression codecs to fail a read eagerly during 
Thrift parsing.
   
   ### Are these changes tested?
   Yes, new test case added to `metadata_test.cc`
   
   ### Are there any user-facing changes?
   No
   
   ### Was AI used for this PR?
   **PR code and description written by:**
   
   - [ ] Human
   - [ ] AI
   
   **Reviewed before submission by:**
   
   - [ ] Human
   - [ ] AI
   - [ ] Not reviewed
   
   Closes #53518 


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