Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve DataFusion error handling #7360

Open
11 of 23 tasks
comphead opened this issue Aug 21, 2023 · 1 comment
Open
11 of 23 tasks

Improve DataFusion error handling #7360

comphead opened this issue Aug 21, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@comphead
Copy link
Contributor

comphead commented Aug 21, 2023

Is your feature request related to a problem or challenge?

I'm trying to collect known issues related to error handling in datafusion.

So far I can see different categories which can be improved

  1. DataFusion heavily relies on Rust Result<T, E> and sometimes its hard to identify context where the actual error is happening, the situation becomes even worse for downstream projects.
  1. Overuse panics
  1. Overuse internal error type in situations where more specific error kind can be used
  1. General

Describe the solution you'd like

We can try to address some of the issues with implementing a generic error macros that will be used for handling DF errors. Having done that allows to process errors in standard unified way, experiment with stacktraces and other details needed to be exposed for the end user.

Describe alternatives you've considered

No response

Additional context

No response

@alamb
Copy link
Contributor

alamb commented Apr 22, 2024

I think we have made significant progress on this issue. I wonder if we still have more planned work?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants