hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chen He (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HDFS-6888) Remove audit logging of getFIleInfo()
Date Thu, 02 Oct 2014 16:18:34 GMT

     [ https://issues.apache.org/jira/browse/HDFS-6888?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Chen He updated HDFS-6888:
--------------------------
    Attachment: HDFS-6888-5.patch

Thank you for the review, [~eepayne]. I change "outputs" to "writes to" in the latest patch.

For the documentation changes, I am not sure. But, I did some research in the source code:


There are 3 hdfs-default.xml files:
./hadoop-hdfs-project/hadoop-hdfs/src/main/resources/hdfs-default.xml
./hadoop-hdfs-project/hadoop-hdfs/src/site/resources/hdfs-default.xml
./hadoop-hdfs-project/hadoop-hdfs/target/classes/hdfs-default.xml

The 3rd one is from the 1st one (same contents). But it looks like the 2nd one is for webpage
(site package). However, the 2nd one has more 500 lines difference between 1st one. To confirm
which one is used by current hadoop.com, I checked http://hadoop.apache.org/docs/current/hadoop-project-dist/hadoop-common/core-default.xml,
The contents are more closer to the 1st one (or maybe 3rd one). 

Then, I guess that we may not need to modify the 2nd one which is "./hadoop-hdfs-project/hadoop-hdfs/src/site/resources/hdfs-default.xml".


> Remove audit logging of getFIleInfo()
> -------------------------------------
>
>                 Key: HDFS-6888
>                 URL: https://issues.apache.org/jira/browse/HDFS-6888
>             Project: Hadoop HDFS
>          Issue Type: Bug
>    Affects Versions: 2.5.0
>            Reporter: Kihwal Lee
>            Assignee: Chen He
>              Labels: log
>         Attachments: HDFS-6888-2.patch, HDFS-6888-3.patch, HDFS-6888-4.patch, HDFS-6888-5.patch,
HDFS-6888.patch
>
>
> The audit logging of getFileInfo() was added in HDFS-3733.  Since this is a one of the
most called method, users have noticed that audit log is now filled with this.  Since we now
have HTTP request logging, this seems unnecessary.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message