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

   ### Rationale for this change
   
   This makes it so trying to read with a column chunk reader consistently 
returns an error if the index is outside the bounds of the logical fields 
(currently it panics in some cases and returns an error in others).
   
   ### What changes are included in this PR?
   
   This makes it so the column chunk reader checks the number of logical fields 
instead of the number of physical columns when checking if an index is out of 
range.
   
   ### Are these changes tested?
   
   The new test will panics without the accompanying code change.
   
   ### Are there any user-facing changes?
   
   Applications that used to panic will now have an error to handle instead.
   


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