[
https://issues.apache.org/jira/browse/MAPREDUCE-6883?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16002010#comment-16002010
]
Haibo Chen edited comment on MAPREDUCE-6883 at 5/9/17 4:35 AM:
---------------------------------------------------------------
I could not reproduce the test failure locally either. Looks like AuditLogger
is also referenced in hadoop-common/src/main/log4j.properties,
> log4j.logger.org.apache.hadoop.mapred.AuditLogger=${mapred.audit.logger} \
> log4j.additivity.org.apache.hadoop.mapred.AuditLogger=false
can you update the patch to remove those, [~vrushalic]?
was (Author: haibochen):
I could not reproduce the test failure locally either. Looks like AuditLogger
is also referenced in hadoop-common/src/main/log4j.properties,
bq. log4j.logger.org.apache.hadoop.mapred.AuditLogger=${mapred.audit.logger}
log4j.additivity.org.apache.hadoop.mapred.AuditLogger=false
can you update the patch to remove those, [~vrushalic]?
> AuditLogger and TestAuditLogger are dead code
> ---------------------------------------------
>
> Key: MAPREDUCE-6883
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6883
> Project: Hadoop Map/Reduce
> Issue Type: Improvement
> Components: client
> Affects Versions: 2.8.0
> Reporter: Daniel Templeton
> Assignee: Vrushali C
> Priority: Minor
> Labels: newbie
> Attachments: MAPREDUCE-6883.001.patch, MAPREDUCE-6883.002.patch
>
>
> The {{AuditLogger}} and {{TestAuditLogger}} classes appear to be dead code.
> I can't find anything that uses or references {{AuditLogger}}. No one has
> touched the code 2011. I think it's safe to remove.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
