[HADOOP-17001] The suffix name of the unified compression class

• 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
• Target Version/s: 3.2.2
• Hadoop Flags: Reviewed
• Assignee: bianqi
• Reporter: bianqi
• Created: 19/Apr/20 08:04
• Updated: 22/Apr/20 20:27
• Resolved: 22/Apr/20 20:27

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.