Details
-
Improvement
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
None
Description
- 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:
Rationale: don't use panic! directly (though the caller of this function still calls `unwrap()` so I am not sure how much of an improvement this change really is. However it sets us up for a more `safe` future eventually
Attachments
Issue Links
- links to