tustvold commented on issue #5446:
URL: https://github.com/apache/arrow-rs/issues/5446#issuecomment-1972533899

   As you state in your issue, this behaviour is documented and explained 
[here](https://docs.rs/object_store/latest/object_store/path/struct.Path.html#encode),
 it is not a bug and is intentional. The rationale for performing 
percent-encoding is to obtain a safe path that will work across a majority of 
storage backends. This is a common pattern when storing arbitrary data in 
object_stores, and is widely used by systems like Hive.
   
   > Path::from is inconsistent with the other APIs
   
   We by default encode because it is guaranteed to infallibly yield a valid 
`Path` that will work with all stores. We do, however, provide fallible 
conversions from various other path representations that may be more or less 
opinionated.
   
   Whilst I recognise opinions may differ on the validity of this approach, 
ultimately this is the approach that this crate adopted, and it would be 
extremely disruptive to revisit this now. On a related note, I might suggest 
that filing an issue attesting a clearly documented behaviour to actually be a 
bug, is not a fantastic way to engender support for your proposal.


-- 
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]

Reply via email to