tustvold commented on PR #5616: URL: https://github.com/apache/arrow-rs/pull/5616#issuecomment-2045275694
> I thought the current field() was kind of an "unsafe" function I think I would prefer to keep things as they are, I don't see an issue with panicking when an invariant is violated, especially as you get a helpful backtrace to the point of issue. I personally don't subscribe to the notion that panics are some evil thing to be avoided, they're just exceptions and they avoid literring code with unnecessary error handling verbosity. Given there are simple alternatives for people who want to avoid panics for whatever ideological reasons, I think we're ok? -- 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]
