jorisvandenbossche commented on code in PR #13564:
URL: https://github.com/apache/arrow/pull/13564#discussion_r925213050


##########
python/pyarrow/_fs.pyx:
##########
@@ -335,6 +432,22 @@ cdef class FileSystem(_Weakrefable):
         tuple of (FileSystem, str path)
             With (filesystem, path) tuple where path is the abstract path
             inside the FileSystem instance.
+
+        Examples
+        --------
+        Create a new FileSystem subclass from a URI:
+
+        >>> uri = 'file:///{}/pyarrow-fs-example.dat'.format(local_path)

Review Comment:
   Actually, we don't need to format it with a "local_path" variable, as this 
URI can be any random path (the method doesn't verify that the path exists, 
only parses it):
   
   ```
   In [34]: fs.FileSystem.from_uri("file:///home/user/example.dat")
   Out[34]: (<pyarrow._fs.LocalFileSystem at 0x7f269d9cc530>, 
'/home/user/example.dat')
   ```



-- 
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-unsubscr...@arrow.apache.org

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

Reply via email to