sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cohei...@apache.org
Subject sentry git commit: Some trivial logging changes
Date Tue, 27 Sep 2016 15:03:45 GMT
Repository: sentry
Updated Branches:
  refs/heads/master 164ea5f1a -> cbc7efcfb


Some trivial logging changes


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

Branch: refs/heads/master
Commit: cbc7efcfb9cc8d254a4733dc07385d1ef49d756d
Parents: 164ea5f
Author: Colm O hEigeartaigh <coheigea@apache.org>
Authored: Tue Sep 27 16:03:15 2016 +0100
Committer: Colm O hEigeartaigh <coheigea@apache.org>
Committed: Tue Sep 27 16:03:15 2016 +0100

----------------------------------------------------------------------
 .../src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java | 2 +-
 .../sentry/provider/common/ResourceAuthorizationProvider.java      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/cbc7efcf/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java
----------------------------------------------------------------------
diff --git a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java
b/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java
index a0199fa..c24a6cd 100644
--- a/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java
+++ b/sentry-core/sentry-core-common/src/main/java/org/apache/sentry/core/common/ActiveRoleSet.java
@@ -74,6 +74,6 @@ public class ActiveRoleSet {
     } else {
       builder.append(roles);
     }
-    return builder.append(" ").toString();
+    return builder.append(" ] ").toString();
   }
 }

http://git-wip-us.apache.org/repos/asf/sentry/blob/cbc7efcf/sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
b/sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
index a6b2047..1ad292e 100644
--- a/sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
+++ b/sentry-provider/sentry-authorization-provider/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
@@ -117,7 +117,7 @@ public abstract class ResourceAuthorizationProvider implements AuthorizationProv
          */
         boolean result = permission.implies(priv, model);
         if (LOGGER.isDebugEnabled()) {
-          LOGGER.debug("ProviderPrivilege {}, RequestPrivilege {}, RoleSet, {}, Result {}",
+          LOGGER.debug("ProviderPrivilege {}, RequestPrivilege {}, RoleSet {}, Result {}",
               new Object[]{ permission, requestPrivilege, roleSet, result});
         }
         if (result) {


Mime
View raw message