You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by co...@apache.org on 2016/09/27 15:03:45 UTC
sentry git commit: Some trivial logging changes
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 <co...@apache.org>
Authored: Tue Sep 27 16:03:15 2016 +0100
Committer: Colm O hEigeartaigh <co...@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) {