tustvold commented on issue #3651: URL: https://github.com/apache/arrow-rs/issues/3651#issuecomment-1414512964
> can support receiving encoded path Adding a `Path::from_url_path` that accepts a `Url` argument makes sense to me, I would be happy to review a PR that adds this :+1: This would still need to decode the path, in order to ensure [path safety](https://docs.rs/object_store/latest/object_store/path/struct.Path.html#path-safety), but would save users that complexity. FWIW overheads of encoding and [signing](https://docs.aws.amazon.com/general/latest/gr/create-signed-request.html#create-canonical-request) the requests will dominate any additional latency resulting from this encoding dance, and both will be completely dominated by the latency of the store itself (typically in the 10s or 100s of milliseconds). -- 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]
