You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by ak...@apache.org on 2017/09/11 22:46:13 UTC

sentry git commit: SENTRY-1932: Improve logging for HMSPath (Arjun Mishra, reviewed by Brian Towles and Alex Kolbasov)

Repository: sentry
Updated Branches:
  refs/heads/master b75b9d968 -> bbf5ce1fd


SENTRY-1932: Improve logging for HMSPath (Arjun Mishra, reviewed by Brian Towles and Alex Kolbasov)


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

Branch: refs/heads/master
Commit: bbf5ce1fd7f4cc57af6bf8291c2455685a0e33f7
Parents: b75b9d9
Author: Alexander Kolbasov <ak...@cloudera.com>
Authored: Mon Sep 11 15:44:04 2017 -0700
Committer: Alexander Kolbasov <ak...@cloudera.com>
Committed: Mon Sep 11 15:44:04 2017 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/sentry/hdfs/HMSPaths.java          | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/bbf5ce1f/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java
----------------------------------------------------------------------
diff --git a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java
index 5a98263..5e975cb 100644
--- a/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java
+++ b/sentry-hdfs/sentry-hdfs-common/src/main/java/org/apache/sentry/hdfs/HMSPaths.java
@@ -421,8 +421,7 @@ public class HMSPaths implements AuthzPaths {
         // we only create the entry if is under a prefix, else we ignore it
         entry = createChild(pathElements, EntryType.AUTHZ_OBJECT, authzObj);
       } else {
-        LOG.info("Skipping to create authzObjPath as it is outside of prefix. authObj = " + authzObj
-        + " pathElements=" + pathElements);
+        LOG.debug("Skipping to create authzObjPath as it is outside of prefix. authObj={} pathElements={}", authzObj, pathElements);
       }
       return entry;
     }
@@ -702,7 +701,7 @@ public class HMSPaths implements AuthzPaths {
         if (e != null) {
           newEntries.add(e);
         } else {
-          LOG.info("Path outside prefix");
+          LOG.debug("Cannot create authz obj path for {} because it is outside of prefix", authzObj);
         }
       }
       entries.addAll(newEntries);