[jira] [Updated] (HADOOP-17001) The suffix name of the unified compression class

["bianqi (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/HADOOP-17001?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

bianqi updated HADOOP-17001:
----------------------------
    Attachment: HADOOP-17001-001.patch

> 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
>            Priority: Major
>             Fix For: 3.2.2
>
>         Attachments: HADOOP-17001-001.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-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org