You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by eb...@apache.org on 2021/02/09 19:36:38 UTC

[hadoop] branch branch-2.10 updated: MAPREDUCE-7319. Log list of mappers at trace level in ShuffleHandler audit log. Contributed by Jim Brennan.

This is an automated email from the ASF dual-hosted git repository.

ebadger pushed a commit to branch branch-2.10
in repository https://gitbox.apache.org/repos/asf/hadoop.git


The following commit(s) were added to refs/heads/branch-2.10 by this push:
     new 585d285  MAPREDUCE-7319. Log list of mappers at trace level in ShuffleHandler audit log. Contributed by Jim Brennan.
585d285 is described below

commit 585d285e1d7a373173b5aa80da1de58605346ae7
Author: Eric Badger <eb...@verizonmedia.com>
AuthorDate: Tue Feb 9 19:29:22 2021 +0000

    MAPREDUCE-7319. Log list of mappers at trace level in ShuffleHandler audit log.
    Contributed by Jim Brennan.
    
    (cherry picked from commit 462561654bb37dc1d453bfe15c3e75c274bbfb02)
---
 .../src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java   | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java
index f2431f2..fe480f8 100644
--- a/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java
+++ b/hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-shuffle/src/main/java/org/apache/hadoop/mapred/ShuffleHandler.java
@@ -1175,8 +1175,13 @@ public class ShuffleHandler extends AuxiliaryService {
         StringBuilder sb = new StringBuilder("shuffle for ");
         sb.append(jobId).append(" reducer ").append(reduce);
         sb.append(" length ").append(contentLength);
-        sb.append(" mappers: ").append(mapIds);
-        AUDITLOG.debug(sb.toString());
+        if (AUDITLOG.isTraceEnabled()) {
+          // For trace level logging, append the list of mappers
+          sb.append(" mappers: ").append(mapIds);
+          AUDITLOG.trace(sb.toString());
+        } else {
+          AUDITLOG.debug(sb.toString());
+        }
       }
     }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org