sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ak...@apache.org
Subject [1/2] sentry git commit: Some trivial logging changes
Date Sat, 11 Mar 2017 02:17:15 GMT
Repository: sentry
Updated Branches:
  refs/heads/sentry-ha-redesign f2176bbf6 -> 3cbaa91cf


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/5994ec3a
Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/5994ec3a
Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/5994ec3a

Branch: refs/heads/sentry-ha-redesign
Commit: 5994ec3a1726754b659eee3d7d2eac308b5eada6
Parents: f2176bb
Author: Alexander Kolbasov <akolb@cloudera.com>
Authored: Fri Mar 10 18:15:20 2017 -0800
Committer: Alexander Kolbasov <akolb@cloudera.com>
Committed: Fri Mar 10 18:15:20 2017 -0800

----------------------------------------------------------------------
 .../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/5994ec3a/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/5994ec3a/sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
b/sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
index 4e22071..005724f 100644
--- a/sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
+++ b/sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java
@@ -116,7 +116,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