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 <[email protected]> Authored: Tue Jul 21 09:42:50 2015 +0800 Committer: Colin Ma <[email protected]> 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() + "]"); }
