You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/30 08:33:25 UTC

[GitHub] [arrow-datafusion] crepererum opened a new issue, #4434: Do NOT convert intermediate/source errors to strings.

crepererum opened a new issue, #4434:
URL: https://github.com/apache/arrow-datafusion/issues/4434

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   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).
   
   **Describe the solution you'd like**
   There are a few places where we convert errors to strings, mostly to wrap them into `DataFusionError::Execution(...)`. However DF (and Arrow) have ways to wrap arbitrary error types (external error). To send errors to multiple receivers, they can easily be wrapped into `Arc`s (`Arc` forwards `std::error::Error`).
   
   **Describe alternatives you've considered**
   Perform string "parsing" on the top-level `err.to_string()`. This is pretty bad code smell.
   
   **Additional context**
   See #3940 for an application.
   


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] crepererum commented on issue #4434: Do NOT convert intermediate/source errors to strings.

Posted by GitBox <gi...@apache.org>.
crepererum commented on issue #4434:
URL: https://github.com/apache/arrow-datafusion/issues/4434#issuecomment-1331805021

   I'm working on this.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [arrow-datafusion] alamb closed issue #4434: Do NOT convert intermediate/source errors to strings.

Posted by GitBox <gi...@apache.org>.
alamb closed issue #4434: Do NOT convert intermediate/source errors to strings.
URL: https://github.com/apache/arrow-datafusion/issues/4434


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org