pitrou commented on a change in pull request #8149: URL: https://github.com/apache/arrow/pull/8149#discussion_r500982254
########## File path: python/pyarrow/__init__.py ########## @@ -190,23 +191,58 @@ def show_versions(): SerializationCallbackError, DeserializationCallbackError) -from pyarrow.filesystem import FileSystem, LocalFileSystem - -from pyarrow.hdfs import HadoopFileSystem import pyarrow.hdfs as hdfs from pyarrow.ipc import serialize_pandas, deserialize_pandas import pyarrow.ipc as ipc - -localfs = LocalFileSystem.get_instance() - from pyarrow.serialization import (default_serialization_context, register_default_serialization_handlers, register_torch_serialization_handlers) import pyarrow.types as types + +# deprecated filesystems + +from pyarrow.filesystem import FileSystem as _FileSystem, LocalFileSystem as _LocalFileSystem +from pyarrow.hdfs import HadoopFileSystem as _HadoopFileSystem + +_localfs = _LocalFileSystem._get_instance() + + +_msg = "pyarrow.{0} is deprecated as of 2.0.0, please use pyarrow.fs.{1} instead." + + +if _sys.version_info >= (3, 7): + def __getattr__(name): + if name == "localfs": Review comment: Looks like you want to write a loop and/or helper function to avoid all this copy/pasting? ---------------------------------------------------------------- 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