westonpace opened a new issue, #35399:
URL: https://github.com/apache/arrow/issues/35399
### Describe the enhancement requested
This is a fairly minor change but it is something I encountered trying to
simplify the implementation of these methods in #34420. Currently, these
methods may or may not remove the trailing slash from a path. For example:
```
FileSystemFromUriOrPath("s3://foo/bar/", &out_path);
FileSystemFromUriOrPath("file:///foo/bar/", &out_path);
```
The former returns `foo/bar` and the latter returns `/foo/bar/`. I think
the change leading slash being different between the two is ok. However, I
think there is no reason to remove the trailing slash in one instance and not
the other.
No work needs to be done here, I'm addressing this already in #34420 .
However, I wanted to open a new issue to ensure visibility in case someone was
relying on the old behavior or knew of a reason that the old behavior existed.
### Component(s)
C++
--
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]
