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

Hadoop QA commented on MAPREDUCE-5981:
--------------------------------------

\\
\\
| (x) *{color:red}-1 overall{color}* |
\\
\\
|| Vote || Subsystem || Runtime || Comment ||
| {color:red}-1{color} | patch |   0m  0s | The patch command could not apply 
the patch during dryrun. |
\\
\\
|| Subsystem || Report/Notes ||
| Patch URL | 
http://issues.apache.org/jira/secure/attachment/12656504/MAPREDUCE-5981.patch |
| Optional Tests | javadoc javac unit findbugs checkstyle |
| git revision | trunk / f1a152c |
| Console output | 
https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/5568/console |


This message was automatically generated.

> Log levels of certain MR logs can be changed to DEBUG
> -----------------------------------------------------
>
>                 Key: MAPREDUCE-5981
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-5981
>             Project: Hadoop Map/Reduce
>          Issue Type: Improvement
>            Reporter: Varun Saxena
>            Assignee: Varun Saxena
>         Attachments: MAPREDUCE-5981.patch
>
>
> Following map reduce logs can be changed to DEBUG log level.
> 1. In 
> org.apache.hadoop.mapreduce.task.reduce.Fetcher#copyFromHost(Fetcher.java : 
> 313), the second log is not required to be at info level. This can be moved 
> to debug as a warn log is anyways printed if verifyReply fails.
>       SecureShuffleUtils.verifyReply(replyHash, encHash, shuffleSecretKey);
>       LOG.info("for url="+msgToEncode+" sent hash and received reply");
> 2. Thread related info need not be printed in logs at INFO level. Below 2 
> logs can be moved to DEBUG
>     a) In 
> org.apache.hadoop.mapreduce.task.reduce.ShuffleSchedulerImpl#getHost(ShuffleSchedulerImpl.java
>  : 381), below log can be changed to DEBUG
>    LOG.info("Assigning " + host + " with " + host.getNumKnownMapOutputs() +
>                " to " + Thread.currentThread().getName());
>     b) In 
> org.apache.hadoop.mapreduce.task.reduce.ShuffleScheduler.getMapsForHost(ShuffleSchedulerImpl.java
>  : 411), below log can be changed to DEBUG
>  LOG.info("assigned " + includedMaps + " of " + totalSize + " to " +
>              host + " to " + Thread.currentThread().getName());
>  



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

Reply via email to