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

The suffix name of the unified compression class

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 3.2.1
    • 3.3.0, 3.1.4, 3.2.2, 3.4.0
    • io
    • None
    • Reviewed

    Description

      The suffix name of the unified compression class,I think the suffix name in the compression class should be extracted into a constant class, which is helpful for developers to understand the structure of the compression class as a whole.

      public static final String OPT_EXTENSION =
      "io.compress.passthrough.extension";

      /**

      • This default extension is here so that if no extension has been defined,
      • some value is still returned: {@value}..
        */
        public static final String DEFAULT_EXTENSION = ".passthrough";

      private Configuration conf;

      private String extension = DEFAULT_EXTENSION;

      public PassthroughCodec() {
      }

      The above code, the use of constants is a bit messy.

      Attachments

        1. HADOOP-17001.005.patch
          7 kB
          bianqi
        2. HADOOP-17001.004.patch
          7 kB
          bianqi
        3. HADOOP-17001.003.patch
          7 kB
          bianqi

        Activity

          People

            bianqi bianqi
            bianqi bianqi
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: