You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "westonpace (via GitHub)" <gi...@apache.org> on 2023/05/03 02:10:23 UTC

[GitHub] [arrow] westonpace commented on pull request #34420: GH-34386: [C++] Add a PathFromUriOrPath method

westonpace commented on PR #34420:
URL: https://github.com/apache/arrow/pull/34420#issuecomment-1532372839

   I unified the various paths using a helper method in util_internal.cc.  This highlighted one small difference between the various implementations.  The s3 and gcs filesystems remove any trailing slash in the path while the hdfs and local filesystems do not.  I have opened #35399 for any discussion on that difference but, for the moment, I have changed all the filesystems to always remove the trailing slash for consistency.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org