You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by sd...@apache.org on 2015/08/14 09:28:54 UTC
[20/50] [abbrv] incubator-sentry git commit: SENTRY-796: Fix log
levels in SentryAuthorizationInfo (Colin Ma, Reviewed by: Dapeng Sun)
SENTRY-796: Fix log levels in SentryAuthorizationInfo (Colin Ma, Reviewed by: Dapeng Sun)
Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/412eea34
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/412eea34
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/412eea34
Branch: refs/heads/hive_plugin_v2
Commit: 412eea346dea9c65866fc4cbf0a88df21250a598
Parents: f5445bb
Author: Colin Ma <co...@apache.org>
Authored: Tue Jul 21 09:42:50 2015 +0800
Committer: Colin Ma <co...@apache.org>
Committed: Tue Jul 21 09:42:50 2015 +0800
----------------------------------------------------------------------
.../org/apache/sentry/hdfs/SentryAuthorizationInfo.java | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/412eea34/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java b/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java
index d178c3e..c9accc1 100644
--- a/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java
+++ b/sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java
@@ -31,7 +31,6 @@ import org.apache.hadoop.conf.Configuration;
import org.apache.hadoop.fs.Path;
import org.apache.hadoop.fs.permission.AclEntry;
import org.apache.hadoop.util.StringUtils;
-import org.apache.sentry.hdfs.SentryAuthzUpdate;
import org.apache.sentry.hdfs.Updateable.Update;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -139,11 +138,11 @@ public class SentryAuthorizationInfo implements Runnable {
if ((newAuthzPaths != authzPaths)||(newAuthzPerms != authzPermissions)) {
lock.writeLock().lock();
try {
- LOG.warn("FULL Updated paths seq Num [old="
+ LOG.debug("FULL Updated paths seq Num [old="
+ authzPaths.getLastUpdatedSeqNum() + "], [new="
+ newAuthzPaths.getLastUpdatedSeqNum() + "]");
authzPaths = newAuthzPaths;
- LOG.warn("FULL Updated perms seq Num [old="
+ LOG.debug("FULL Updated perms seq Num [old="
+ authzPermissions.getLastUpdatedSeqNum() + "], [new="
+ newAuthzPerms.getLastUpdatedSeqNum() + "]");
authzPermissions = newAuthzPerms;
@@ -162,20 +161,20 @@ public class SentryAuthorizationInfo implements Runnable {
// one in the List.. all the remaining will be partial updates
if (updates.size() > 0) {
if (updates.get(0).hasFullImage()) {
- LOG.warn("Process Update : FULL IMAGE "
+ LOG.debug("Process Update : FULL IMAGE "
+ "[" + updateable.getClass() + "]"
+ "[" + updates.get(0).getSeqNum() + "]");
updateable = (V)updateable.updateFull(updates.remove(0));
}
// Any more elements ?
if (!updates.isEmpty()) {
- LOG.warn("Process Update : More updates.. "
+ LOG.debug("Process Update : More updates.. "
+ "[" + updateable.getClass() + "]"
+ "[" + updateable.getLastUpdatedSeqNum() + "]"
+ "[" + updates.size() + "]");
updateable.updatePartial(updates, lock);
}
- LOG.warn("Process Update : Finished updates.. "
+ LOG.debug("Process Update : Finished updates.. "
+ "[" + updateable.getClass() + "]"
+ "[" + updateable.getLastUpdatedSeqNum() + "]");
}