[ https://issues.apache.org/jira/browse/HADOOP-17001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Wei-Chiu Chuang updated HADOOP-17001: ------------------------------------- Fix Version/s: (was: 3.2.2) > The suffix name of the unified compression class > ------------------------------------------------ > > Key: HADOOP-17001 > URL: https://issues.apache.org/jira/browse/HADOOP-17001 > Project: Hadoop Common > Issue Type: Improvement > Components: io > Affects Versions: 3.2.1 > Reporter: bianqi > Assignee: bianqi > Priority: Major > Attachments: HADOOP-17001-001.patch, HADOOP-17001-002.patch > > > 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. > {quote}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() { > } > {quote} > The above code, the use of constants is a bit messy. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org