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

Suresh Srinivas commented on HADOOP-7956:
-----------------------------------------

Eli, the default.xml serves as documentation of config parameters. Currently 
the ones that are not defined in that are hidden configs that used only by 
internal implementation. The defaults in the code currently serves in case 
default.xml gets edited. Though the duplication seems not clean, not sure how 
you can avoid that. Could we throw an exception if the default expected value 
from the config is missing and get rid of the default defined in the code? That 
seems like a lot of work.
                
> Remove duplicate definition of default config values
> ----------------------------------------------------
>
>                 Key: HADOOP-7956
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7956
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Eli Collins
>
> We define default configuration values in two places:
> #1 The default.xml files (eg core-default.xml)
> #2 The _DEFAULT defines in *Keys.java
> This means the defaults used in the code may or may not be dead code based on 
> whether the config is present in the default xml file. Would be good to 
> define these in one place. Eg:
> #1 Just have the defines in the code and figure out how to make those 
> accessible as a loadable resource (eg could generate the default files from 
> the defines in the KeysPublic* files)
> #2 Remove one of the definitions entirely (possible to live w/o the default 
> files?) or
> #3 Remove the overlap between the code and default files

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to