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

Sharad Agarwal commented on HADOOP-4828:
----------------------------------------

bq. The following files has references to hadoop-default and hadoop-site in the 
documentation. src/core/overview.html and 
src/benchmarks/gridmix2/README.gridmix2
could not find any references of hadoop-default/hadoop-site in 
src/core/overview.html. the current patch takes care of them.
src/benchmarks/gridmix2/README.gridmix2 contains references to hadoop-site.xml. 
I think it is fine as we are still supporting the hadoop-site.xml

bq. src/test, src/c++/libhdfs/tests/conf/ and src/contrib/test/ directories 
contains a file named hadoop-site.xml. I think this can be removed
The reason to keep the empty hadoop-site.xml files is to *avoid* picking 
properties from conf/hadoop-site.xml. The test cases have conf  also in 
classpath, so if an user has hadoop-site.xml in conf directory, the properties 
from it gets picked up. To override that bahaviour the empty hadoop-site.xml in 
test config folder is put. This would be removed once we remove the deprecated 
support for hadoop-site.xml.

bq. The following files are still using hadoop-default and hadoop-site as the 
conf resources :
I think it is ok to refer hadoop-site.xml as we support it now. hadoop-default 
should not be referred. I found only following two to refer hadoop-default:
src/contrib/failmon/src/java/org/apache/hadoop/contrib/failmon/HDFSMerger.java
src/contrib/failmon/src/java/org/apache/hadoop/contrib/failmon/LocalStore.java
This should perhaps be fixed in another jira as it won't be a documentation 
change. btw it is a minor change and doesn't affect the functionality.

bq. Also, the chinese docs have references in the following files:
Will file another jira for this.

bq. Hod documentation still has references to hadoop-site.xml
Currently HOD generate the single configuration file as hadoop-site.xml and the 
framework supports that. So the documentation will still refer to it. This will 
be changed once HOD move to the new configuration structure.

> Update documentation for default configuration
> ----------------------------------------------
>
>                 Key: HADOOP-4828
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4828
>             Project: Hadoop Core
>          Issue Type: Improvement
>          Components: documentation
>            Reporter: Sharad Agarwal
>            Assignee: Sharad Agarwal
>             Fix For: 0.20.0
>
>         Attachments: 4828_v1.patch
>
>
> Documentation needs to be updated as per the configuration changes in 
> HADOOP-4631

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to