You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by sn...@apache.org on 2014/10/23 23:30:17 UTC

git commit: ARGUS-129: modified debug messages to be logged with debug level instead of info level

Repository: incubator-argus
Updated Branches:
  refs/heads/master dd87ada1b -> e57ee6c51


ARGUS-129: modified debug messages to be logged with debug level instead of info level


Project: http://git-wip-us.apache.org/repos/asf/incubator-argus/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-argus/commit/e57ee6c5
Tree: http://git-wip-us.apache.org/repos/asf/incubator-argus/tree/e57ee6c5
Diff: http://git-wip-us.apache.org/repos/asf/incubator-argus/diff/e57ee6c5

Branch: refs/heads/master
Commit: e57ee6c51dfd6c309ec0b77d3e87aa3ab61bd9ae
Parents: dd87ada
Author: sneethiraj <sn...@apache.org>
Authored: Thu Oct 23 14:21:03 2014 -0400
Committer: sneethiraj <sn...@apache.org>
Committed: Thu Oct 23 14:21:03 2014 -0400

----------------------------------------------------------------------
 .../storm/authorizer/XaSecureStormAuthorizer.java       | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-argus/blob/e57ee6c5/storm-agent/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java
----------------------------------------------------------------------
diff --git a/storm-agent/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java b/storm-agent/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java
index b07769c..8366f8e 100644
--- a/storm-agent/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java
+++ b/storm-agent/src/main/java/com/xasecure/authorization/storm/authorizer/XaSecureStormAuthorizer.java
@@ -174,11 +174,13 @@ public class XaSecureStormAuthorizer implements IAuthorizer {
 			LOG.error("XaSecureStormAuthorizer found this exception", t);
 		}
 		finally {
-			LOG.info("[req "+ aRequestContext.requestID()+ "] Access "
-	                + " from: [" + aRequestContext.remoteAddress() + "]"
-	                + " user: [" + aRequestContext.principal() + "],"  
-	                + " op:   [" + aOperationName + "],"
-	                + "topology: [" + topologyName + "] => returns [" + accessAllowed + "]") ;
+			if (LOG.isDebugEnabled()) {
+				LOG.debug("[req "+ aRequestContext.requestID()+ "] Access "
+		                + " from: [" + aRequestContext.remoteAddress() + "]"
+		                + " user: [" + aRequestContext.principal() + "],"  
+		                + " op:   [" + aOperationName + "],"
+		                + "topology: [" + topologyName + "] => returns [" + accessAllowed + "]") ;
+			}
 		}
 		
 		return accessAllowed ;