crepererum opened a new pull request, #4436: URL: https://github.com/apache/arrow-datafusion/pull/4436
# Which issue does this PR close? Closes #4434. # Rationale for this change Now that DF emits `ResourceExhausted` errors for out-of-memory situations, it would be nice if the API user could somehow detect this situation (e.g. to convert this to a suitable HTTP status code). # What changes are included in this PR? Convert a bunch of `Execution` errors into typed errors. # Are these changes tested? **:warning: Consensus required prior to merging :warning:** None. I'm not really sure how we want to prevent regressions here. It's just too many errors. I somewhat check-able rule (via some regex script?!) would be: "no `Execution\(.+\.to_string`". Shall I add this to the `lint` CI workflow? # Are there any user-facing changes? Errors types slightly changed. This also changes error texts a bit. -- 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]
