pitrou commented on pull request #10098: URL: https://github.com/apache/arrow/pull/10098#issuecomment-823309218
I'm in support of this. Performance improvements notwithstanding, the main benefits IMHO are: * consistent error handling throughout the codebase, which significantly improves overall usability and approachibility * the return value of a function (`Status` vs. `void`, or `Result<T>` vs. `T`) advertises whether it can raise an error or not, which wasn't the case with the `KernelContext`-bound error status -- 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]
