-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
ARROW-8213: [Python][Dataset] Opening a dataset with a local incorrect path gives confusing error message #6854
Conversation
Well, can you try to fix it in the C++ implementation? :-) Or do you want me to do it? |
I thought you didn't want to deal with non-URIs on the C++ FileSystem level? |
There's a C++ function called |
@jorisvandenbossche Opinions on this? |
Thanks for that fix! Yeah, I think catching "empty scheme" in the error message is the most straightforward way to handle it |
I don't like this kind of error handling because it is error prone, bound to an error message instead of an error type or kind - I'm fine with it as a workaround though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, this seems okay to me, we can iterate/improve this logic surely
Workaround until it is properly handled in the C++ implementation.