alamb opened a new pull request #9691:
URL: https://github.com/apache/arrow/pull/9691


   # Background:
   Left over cleanups suggested by from @sunchao on  
https://github.com/apache/arrow/pull/9425
   
   Broken out from https://github.com/apache/arrow/pull/9508
   
   # Rationale:
   
   Don't use panic! directly. However,  since the caller of this function still 
calls `unwrap()`, I am not sure how much of an improvement this change really 
is. However it may set us up for a more `safe` future eventually
   


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

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


Reply via email to