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

Chen Liang updated HADOOP-13658:
--------------------------------
    Description: 
In Hadoop Common, there are several places where the config keys are used by 
the literal strings instead of their names as in configuration key classes. The 
default values have the same issue. For example

{code:title=in o.a.h.i.f.t.Compression.java}
conf.setInt("io.compression.codec.lzo.buffersize", 64 * 1024);
{code}

should be

{code}
conf.setInt(
    CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_KEY,
    CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_DEFAULT);
{code}

instead

  was:
In Hadoop Common, there are several places where the config keys are used by 
the literal strings instead of their names as in configuration key classes. For 
example

{code:title=in o.a.h.i.f.t.Compression.java}
conf.setInt("io.compression.codec.lzo.buffersize", 64 * 1024);
{code}

should be

{code}
conf.setInt(CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_KEY, 64 
* 1024);
{code}

instead


> Replace config key literal strings with config key names I: hadoop common
> -------------------------------------------------------------------------
>
>                 Key: HADOOP-13658
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13658
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Chen Liang
>            Assignee: Chen Liang
>            Priority: Minor
>         Attachments: HADOOP-13658.001.patch, HADOOP-13658.002.patch
>
>
> In Hadoop Common, there are several places where the config keys are used by 
> the literal strings instead of their names as in configuration key classes. 
> The default values have the same issue. For example
> {code:title=in o.a.h.i.f.t.Compression.java}
> conf.setInt("io.compression.codec.lzo.buffersize", 64 * 1024);
> {code}
> should be
> {code}
> conf.setInt(
>     CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_KEY,
>     CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_DEFAULT);
> {code}
> instead



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to