[GitHub] [arrow] pitrou commented on a change in pull request #10045: ARROW-12396: [Python][Docs] Clarify serialization/filesystem docstrings about deprecated status

[GitBox <gi...@apache.org>]

pitrou commented on a change in pull request #10045:
URL: https://github.com/apache/arrow/pull/10045#discussion_r613984909



##########
File path: python/pyarrow/hdfs.py
##########
@@ -31,6 +31,10 @@ class HadoopFileSystem(lib.HadoopFileSystem, FileSystem):
     FileSystem interface for HDFS cluster.
 
     See pyarrow.hdfs.connect for full connection details
+
+    .. deprecated:: 2.0
+        `pyarrow.hdfs.HadoopFileSystem` is deprecated,
+        please use `pyarrow.fs.HadoopFileSystem instead.

Review comment:
       Missing the closing quote here.

##########
File path: python/pyarrow/hdfs.py
##########
@@ -184,13 +188,19 @@ def _libhdfs_walk_files_dirs(top_path, contents):
 def connect(host="default", port=0, user=None, kerb_ticket=None,
             extra_conf=None):
     """
-    Connect to an HDFS cluster. All parameters are optional and should
-    only be set if the defaults need to be overridden.
+    DEPRECATED: Connect to an HDFS cluster.
+
+    All parameters are optional and should only be set if the defaults need
+    to be overridden.
 
     Authentication should be automatic if the HDFS cluster uses Kerberos.
     However, if a username is specified, then the ticket cache will likely
     be required.
 
+    .. deprecated:: 2.0
+        `pyarrow.hdfs.connect` is deprecated,
+        please use `pyarrow.fs.HadoopFileSystem instead.

Review comment:
       And here as well.




-- 
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:
users@infra.apache.org