Alejandro Abdelnur commented on HADOOP-4416:
--------------------------------------------
This is not really an instance of HADOOP-4212. Personally I would agree that for configuration values you can safely trim them, but I can see people opposing to it.
For class names it is different, you don't want the spaces.
And it should be done in the {{getClassByName()}} method to properly handle the {{getClasses()}} method, this is important for values like:
{code}
<property>
<name>io.compression.codecs</name>
<value>org.apache.hadoop.io.compress.DefaultCodec,org.apache.hadoop.io.compress.GzipCodec,org.apache.hadoop.io.compress.BZip2Codec</value>
</property>
{code}
Where if you have several classes you may want to have an enter between commas for readability.
class names in Configuration are not resolved if the configuration value has a white space
-------------------------------------------------------------------------------------------
Key: HADOOP-4416
URL: https://issues.apache.org/jira/browse/HADOOP-4416
Project: Hadoop Core
Issue Type: Bug
Environment: all
Reporter: Alejandro Abdelnur
Fix For: 0.19.0
If an entry in the configuration used for a class contains spaces or enters before or after the {{getClass}} and {{getClasses}} methods fail to resolve the class.
For example:
{code}
<property>
<name>mapred.mapper.class</name>
<value>
com.foo.MyMapper
</value>
</property>
{code}
---------------------------------------------------------------------------------------------
Key: HADOOP-4416
URL: https://issues.apache.org/jira/browse/HADOOP-4416
Project: Hadoop Core
Issue Type: Bug
Environment: all
Reporter: Alejandro Abdelnur
Fix For: 0.19.0
If an entry in the configuration used for a class contains spaces or enters before or after the {{getClass}} and {{getClasses}} methods fail to resolve the class.
For example:
{code}
<property>
<name>mapred.mapper.class</name>
<value>
com.foo.MyMapper
</value>
</property>
{code}
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.