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

The suffix name of the unified compression class

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.2.1
    • Fix Version/s: 3.3.0, 3.1.4, 3.2.2, 3.4.0
    • Component/s: io
    • Labels:
      None
    • Target Version/s:
    • Hadoop Flags:
      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.003.patch
          7 kB
          bianqi
        2. HADOOP-17001.004.patch
          7 kB
          bianqi
        3. HADOOP-17001.005.patch
          7 kB
          bianqi

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: