Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-12671

Inconsistent s3a configuration values and incorrect comments

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Patch Available
    • Major
    • Resolution: Unresolved
    • 2.7.1, 2.6.2
    • None
    • conf, documentation, fs/s3
    • None

    Description

      The two values in core-default.xml are wrong.
      fs.s3a.multipart.purge.age
      fs.s3a.connection.timeout
      fs.s3a.connection.establish.timeout

      1. fs.s3a.multipart.purge.age
      (in both 2.6.2 and 2.7.1)
      In core-default.xml , the value is 86400 (24 hours), while in the code it is 14400 (4 hours).

      2. fs.s3a.connection.timeout
      (only appear in 2.6.2)
      In core-default.xml (2.6.2) , the value is 5000, while in the code it is 50000.

        // seconds until we give up on a connection to s3
        public static final String SOCKET_TIMEOUT = "fs.s3a.connection.timeout";
        public static final int DEFAULT_SOCKET_TIMEOUT = 50000;
      


      3. fs.s3a.connection.establish.timeout
      (only appear in 2.7.1)
      In core-default.xml (2.7.1), the value is 5000, while in the code it is 50000.

        // seconds until we give up trying to establish a connection to s3
        public static final String ESTABLISH_TIMEOUT = "fs.s3a.connection.establish.timeout";
        public static final int DEFAULT_ESTABLISH_TIMEOUT = 50000;
      


      btw, the code comments are wrong! The two parameters are in the unit of milliseconds instead of seconds...

      -  // seconds until we give up on a connection to s3
      +  // milliseconds until we give up on a connection to s3
      ...
      -  // seconds until we give up trying to establish a connection to s3
      +  // milliseconds until we give up trying to establish a connection to s3
      

      Attachments

        1. HADOOP-12671.000.patch
          2 kB
          Tianyin Xu

        Issue Links

          Activity

            People

              tianyin Tianyin Xu
              tianyin Tianyin Xu
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated: