[
https://issues.apache.org/jira/browse/MAPREDUCE-6354?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14573328#comment-14573328
]
Jason Lowe commented on MAPREDUCE-6354:
---------------------------------------
Thanks for updating the patch, Chang. The debug log should be wrapped in
AUDITLOG.isDebugEnabled() so we don't perform the string processing if we
aren't going to log it.
Also, even though this wasn't technically part of this fix, please go ahead and
fixup the trailing whitespace in log4j.properties that checkstyle is whining
about. Otherwise it will continue to whine for any other patch that happens to
touch nearby lines.
> shuffle handler should log connection info
> ------------------------------------------
>
> Key: MAPREDUCE-6354
> URL: https://issues.apache.org/jira/browse/MAPREDUCE-6354
> Project: Hadoop Map/Reduce
> Issue Type: Bug
> Reporter: Chang Li
> Assignee: Chang Li
> Attachments: MAPREDUCE-6354.2.patch, MAPREDUCE-6354.3.patch,
> MAPREDUCE-6354.4.patch, MAPREDUCE-6354.5.patch, MAPREDUCE-6354.6.patch,
> MAPREDUCE-6354.7.patch, MAPREDUCE-6354.patch
>
>
> currently, shuffle handler only log connection info in debug mode, we want to
> log that info in a more concise way
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
