bianqi created HADOOP-17001:
-------------------------------

             Summary: 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
             Fix For: 3.2.2


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

Reply via email to