tustvold commented on issue #5345:
URL: https://github.com/apache/arrow-rs/issues/5345#issuecomment-1916904880
I don't think we should expose the retry error, but we could conceivably
expose a status method on object_store::Error. However, this would be specific
to HTTP based stores, and wouldn't work for LocalFileSystem.
Some other options might be:
* Add method to object_store that returns std::io::ErrorKind
* Change external variant to std::io::Error
* Add a new variant for permission errors to object_store::Error (and make
the enum non exhaustive at the same time)
Of these the first has the advantage of not being a breaking change, but is
perhaps a little odd. The second would likely be very disruptive. The third
would likely be the most consistent with the current approach, but would be a
breaking change.
To be honest I am not wedded to any particular approach, they're all some
measure of not amazing 😅
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
