hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alejandro Abdelnur (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-10080) Variables in configuration properties are substituted in wrong order of precedence
Date Fri, 01 Nov 2013 16:02:19 GMT

    [ https://issues.apache.org/jira/browse/HADOOP-10080?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13811385#comment-13811385
] 

Alejandro Abdelnur commented on HADOOP-10080:
---------------------------------------------

I hear you, but I'm afraid this is a huge incompatible change in behavior that would break
things within Hadoop and outside that rely on this behavior. Given that I would close as WONT
FIX.

> Variables in configuration properties are substituted in wrong order of precedence
> ----------------------------------------------------------------------------------
>
>                 Key: HADOOP-10080
>                 URL: https://issues.apache.org/jira/browse/HADOOP-10080
>             Project: Hadoop Common
>          Issue Type: Bug
>          Components: conf
>    Affects Versions: 2.2.0
>            Reporter: Sebastian Nagel
>            Priority: Minor
>
> API doc of [Configuration|https://hadoop.apache.org/docs/stable/api/org/apache/hadoop/conf/Configuration.html]
states:
> {quote}
> Value strings are first processed for variable expansion. The available properties are:
> 1.  Other properties defined in this Configuration; and, if a name is undefined here,
> 2.  Properties in System.getProperties().
> {quote}
> The current implementation of variable expansion, however, gives precedence to system
properties. If a system property is defined it's value is used no matter whether a configuration
property of same name exists.
> This may cause inconsistent behaviour, e.g., if in the API doc example "basedir" is set
as system property ({{-Dbasedir=/mnt/tmp}}) the results of two intuitively equivalent calls
differ:
> {code}
>  new Path(conf.get("tempdir"))        // = /mnt/data/tmp
>  new Path(conf.get("basedir"), "tmp") // = /user/wastl/tmp
> {code}



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Mime
View raw message