svn commit: r1136725 - in /hadoop/common/branches/branch-0.20-security-204: CHANGES.txt src/core/org/apache/hadoop/ipc/Server.java

[omalley]

Author: omalley
Date: Thu Jun 16 22:54:20 2011
New Revision: 1136725

URL: http://svn.apache.org/viewvc?rev=1136725&view=rev
Log:
HADOOP-7272. Remove unnecessary security related info logs. (suresh)

Modified:
    hadoop/common/branches/branch-0.20-security-204/CHANGES.txt   (contents, 
props changed)
    
hadoop/common/branches/branch-0.20-security-204/src/core/org/apache/hadoop/ipc/Server.java

Modified: hadoop/common/branches/branch-0.20-security-204/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-204/CHANGES.txt?rev=1136725&r1=1136724&r2=1136725&view=diff
==============================================================================
--- hadoop/common/branches/branch-0.20-security-204/CHANGES.txt (original)
+++ hadoop/common/branches/branch-0.20-security-204/CHANGES.txt Thu Jun 16 
22:54:20 2011
@@ -36,6 +36,8 @@ Release 0.20.204.0 - unreleased
 
     HADOOP-7356. Fix bin/hadoop scripts (eyang via omalley)
 
+    HADOOP-7272. Remove unnecessary security related info logs. (suresh)
+
     MAPREDUCE-2514. Fix typo in TaskTracker ReinitTrackerAction log message.
     (Jonathan Eagles via cdouglas)
 

Propchange: hadoop/common/branches/branch-0.20-security-204/CHANGES.txt
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Thu Jun 16 22:54:20 2011
@@ -1,5 +1,5 @@
 
/hadoop/common/branches/branch-0.20/CHANGES.txt:826138,826568,829987,831184,833001,880632,898713,909245,909723,960946,1044225
-/hadoop/common/branches/branch-0.20-security/CHANGES.txt:1097202,1098837,1100336,1125170,1125587,1125589,1127362,1131277,1131286,1131290,1131299,1131737,1134140
+/hadoop/common/branches/branch-0.20-security/CHANGES.txt:1097202,1098837,1100336,1125139,1125170,1125587,1125589,1127362,1131277,1131286,1131290,1131299,1131737,1134140
 
/hadoop/common/branches/branch-0.20-security-203/CHANGES.txt:1096071,1097012-1099333,1102071,1128115
 
/hadoop/common/branches/branch-0.20-security-205/CHANGES.txt:1132788,1133133,1133274,1133282
 /hadoop/core/branches/branch-0.18/CHANGES.txt:727226

Modified: 
hadoop/common/branches/branch-0.20-security-204/src/core/org/apache/hadoop/ipc/Server.java
URL: 
http://svn.apache.org/viewvc/hadoop/common/branches/branch-0.20-security-204/src/core/org/apache/hadoop/ipc/Server.java?rev=1136725&r1=1136724&r2=1136725&view=diff
==============================================================================
--- 
hadoop/common/branches/branch-0.20-security-204/src/core/org/apache/hadoop/ipc/Server.java
 (original)
+++ 
hadoop/common/branches/branch-0.20-security-204/src/core/org/apache/hadoop/ipc/Server.java
 Thu Jun 16 22:54:20 2011
@@ -1031,12 +1031,16 @@ public abstract class Server {
               null);
         }
         if (saslServer.isComplete()) {
-          LOG.info("SASL server context established. Negotiated QoP is "
-              + saslServer.getNegotiatedProperty(Sasl.QOP));
+          if (LOG.isDebugEnabled()) {
+            LOG.info("SASL server context established. Negotiated QoP is "
+                + saslServer.getNegotiatedProperty(Sasl.QOP));
+          }
           String qop = (String) saslServer.getNegotiatedProperty(Sasl.QOP);
           useWrap = qop != null && !"auth".equalsIgnoreCase(qop);
           user = getAuthorizedUgi(saslServer.getAuthorizationID());
-          LOG.info("SASL server successfully authenticated client: " + user);
+          if (LOG.isDebugEnabled()) {
+            LOG.info("SASL server successfully authenticated client: " + user);
+          }
           rpcMetrics.incrAuthenticationSuccesses();
           AUDITLOG.info(AUTH_SUCCESSFULL_FOR + user);
           saslContextEstablished = true;