Use single constructor path for HadoopFileSystem

Remove from_uri early return and route netloc through the
same kwargs constructor as all other HDFS properties.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: 2 people

pyiceberg/io/pyarrow.py

@@ -582,9 +582,8 @@ def _initialize_hdfs_fs(self, scheme: str, netloc: Optional[str]) -> FileSystem:
         if replication := self.properties.get(HDFS_REPLICATION):
             hdfs_kwargs["replication"] = int(replication)
         if netloc:
-            query = f"?replication={replication}" if replication else ""
-            return HadoopFileSystem.from_uri(f"{scheme}://{netloc}{query}")
-        if host := self.properties.get(HDFS_HOST):
+            hdfs_kwargs["host"] = netloc
+        elif host := self.properties.get(HDFS_HOST):
             hdfs_kwargs["host"] = host
         if port := self.properties.get(HDFS_PORT):
             # port should be an integer type