fsaintjacques commented on a change in pull request #7156:
URL: https://github.com/apache/arrow/pull/7156#discussion_r441740202
##########
File path: python/pyarrow/_dataset.pyx
##########
@@ -43,6 +44,51 @@ def _forbid_instantiation(klass, subclasses_instead=True):
raise TypeError(msg)
+cdef class FileSource:
+
+ cdef:
+ CFileSource wrapped
+
+ def __cinit__(self, file, FileSystem filesystem=None):
+ cdef:
+ shared_ptr[CFileSystem] c_filesystem
+ c_string c_path
+ shared_ptr[CRandomAccessFile] c_file
+ shared_ptr[CBuffer] c_buffer
+
+ if isinstance(file, FileSource):
Review comment:
This is odd. Should you have `_ensure_file_source` like other methods?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
