jerqi commented on code in PR #9569:
URL: https://github.com/apache/gravitino/pull/9569#discussion_r2680700454
##########
catalogs/hadoop-common/src/main/java/org/apache/gravitino/catalog/hadoop/fs/HDFSFileSystemProvider.java:
##########
@@ -39,12 +42,20 @@ public class HDFSFileSystemProvider implements
FileSystemProvider {
public static final String SCHEME_HDFS = "hdfs";
@Override
- public FileSystem getFileSystem(@Nonnull Path path, @Nonnull Map<String,
String> config)
+ public FileSystem getFileSystem(
Review Comment:
I prefer that clients doesn't need to know the ProxyUserHandler. Maybe we
can have an another HDFSFileSystemProvider.
Actually, it's not good point to use the catalog-common in the client, you
don't need to modify this point.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
