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

Error handling #66

Open
johnchronis opened this issue Nov 18, 2020 · 1 comment
Open

Error handling #66

johnchronis opened this issue Nov 18, 2020 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@johnchronis
Copy link
Collaborator

johnchronis commented Nov 18, 2020

Google released a library for returning and handling error messages some time ago. Now that hustle is growing maybe such a solution would be beneficial to make the code cleaner and handling/propagation of errors easier. We had discussed this in the past, but never went through with it (this library was not available at the time).

You can find more information here: https://abseil.io/docs/cpp/guides/status

What do you think? Would such a convention for error handling play nice with arrow?

@johnchronis johnchronis added the enhancement New feature or request label Nov 18, 2020
@kpgaffney
Copy link
Member

Nice idea! Arrow has a status as well. Our error codes could be a superset of Arrow’s. Not sure about compatibility but I’m guessing we could write some utility functions to make it easier to go back and forth between the two types.

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

4 participants