-
Notifications
You must be signed in to change notification settings - Fork 55
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
File not found error surfaces local path instead of cloud path #118
Comments
I agree with this - it's really hard to debug the issue when I get this error. In my case, I'm passing a cloud path to another library, and the error happens inside it - so it's hard to know how to resolve the problem. |
Thanks for the comment @Mchristos, unfortunately our library just can't know what other libraries try to do with our objects so there's not simple way to inject ourselves into these error messages. This is really another instance of #128, which has a complicated set of tradeoffs for the possible approaches. Right now we're leaning towards the experimental work in #322 as the best approach, so I'd be curious if that works in your case. |
I was loading a file from s3 and mistyped the file location. The error I got showed me the local tmp path, which was misleading. I think this should be the standard error you get when you try to do a cloud operation on a file that doesn't exist (something like "head object does not exist") or at least something that includes the cloud path instead of the local path.
DATA_DIR here is an S3Path.
The text was updated successfully, but these errors were encountered: