[HADOOP-13658] Replace config key literal strings with config key names I: hadoop common

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.8.0, 3.0.0-alpha2
• Component/s: conf
• Target Version/s: 2.8.0
• Hadoop Flags: Reviewed
• Assignee: Chen Liang
• Reporter: Chen Liang
• Created: 26/Sep/16 17:33
• Updated: 28/Sep/16 01:50
• Resolved: 28/Sep/16 00:46

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

in o.a.h.i.f.t.Compression.java

conf.setInt("io.compression.codec.lzo.buffersize", 64 * 1024);

should be

conf.setInt(
    CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_KEY,
    CommonConfigurationKeys.IO_COMPRESSION_CODEC_LZO_BUFFERSIZE_DEFAULT);

instead

Issue Links

• is related to: HDFS-10903 Replace config key literal strings with config key names II: hadoop hdfs