Uploaded image for project: 'Hadoop HDFS'
  1. Hadoop HDFS
  2. HDFS-14850

Optimize FileSystemAccessService#getFileSystemConfiguration

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 3.3.0, 3.1.4, 3.2.2
    • httpfs, performance
    • None
    • Reviewed

    Description

       @Override
        public Configuration getFileSystemConfiguration() {
          Configuration conf = new Configuration(true);
          ConfigurationUtils.copy(serviceHadoopConf, conf);
          conf.setBoolean(FILE_SYSTEM_SERVICE_CREATED, true);
      
          // Force-clear server-side umask to make HttpFS match WebHDFS behavior
          conf.set(FsPermission.UMASK_LABEL, "000");
      
          return conf;
        }
      

      As above code,when call FileSystemAccessService#getFileSystemConfiguration,current code new Configuration every time.
      It is not necessary and affects performance. I think it only need to new Configuration in FileSystemAccessService#init once and FileSystemAccessService#getFileSystemConfiguration get it.

      Attachments

        1. HDFS-14850.005.patch
          3 kB
          Lisheng Sun
        2. HDFS-14850.004(2).patch
          3 kB
          Lisheng Sun
        3. HDFS-14850.004.patch
          3 kB
          Lisheng Sun
        4. HDFS-14850.003.patch
          2 kB
          Lisheng Sun
        5. HDFS-14850.002.patch
          2 kB
          Lisheng Sun
        6. HDFS-14850.001.patch
          2 kB
          Lisheng Sun

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            leosun08 Lisheng Sun
            leosun08 Lisheng Sun
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment