zhtttylz commented on code in PR #5660:
URL: https://github.com/apache/hadoop/pull/5660#discussion_r1197715026


##########
hadoop-hdfs-project/hadoop-hdfs-httpfs/src/main/java/org/apache/hadoop/fs/http/client/HttpFSFileSystem.java:
##########
@@ -1756,6 +1762,18 @@ public FileStatus getFileLinkStatus(final Path path) 
throws IOException {
     return status.makeQualified(getUri(), path);
   }
 
+  @Override
+  public FsStatus getStatus(final Path path) throws IOException {
+    Map<String, String> params = new HashMap<>();

Review Comment:
   Path can be null. Please refer to the method for more information
   Filesystem#getStatus
   ```
   /**
      * Returns a status object describing the use and capacity of the
      * filesystem. If the filesystem has multiple partitions, the
      * use and capacity of the partition pointed to by the specified
      * path is reflected.
      * @param p Path for which status should be obtained. null means
      * the default partition.
      * @return a FsStatus object
      * @throws IOException
      *           see specific implementation
      */
     public FsStatus getStatus(Path p) throws IOException {
       return new FsStatus(Long.MAX_VALUE, 0, Long.MAX_VALUE);
     }
   ```



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to