You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by sp...@apache.org on 2017/12/07 16:54:39 UTC

sentry git commit: SENTRY-2092: Drop Role log message shows "Creating role" (Na Li, reviewed by Alexander Kolbasov)

Repository: sentry
Updated Branches:
  refs/heads/master 344be54e8 -> 71daaf037


SENTRY-2092: Drop Role log message shows "Creating role" (Na Li, reviewed by Alexander Kolbasov)


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

Branch: refs/heads/master
Commit: 71daaf0372f8c418ad8035bdc6969e12303aacc4
Parents: 344be54
Author: Sergio Pena <se...@cloudera.com>
Authored: Thu Dec 7 10:53:51 2017 -0600
Committer: Sergio Pena <se...@cloudera.com>
Committed: Thu Dec 7 10:53:51 2017 -0600

----------------------------------------------------------------------
 .../hive/v2/authorizer/DefaultSentryAccessController.java        | 4 ++--
 .../sentry/binding/hive/authz/DefaultSentryAccessController.java | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/71daaf03/sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java
----------------------------------------------------------------------
diff --git a/sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java b/sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java
index fed483f..13ee2cf 100644
--- a/sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java
+++ b/sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/sentry/binding/hive/v2/authorizer/DefaultSentryAccessController.java
@@ -123,7 +123,7 @@ public class DefaultSentryAccessController extends SentryHiveAccessController {
       HiveOperation hiveOp = HiveOperation.CREATEROLE;
       executeOnFailureHooks(hiveOp, e);
     } catch (SentryUserException e) {
-      String msg = "Error occurred when Sentry client creating role: " + e.getMessage();
+      String msg = "Sentry failed to create role: " + e.getMessage();
       executeOnErrorHooks(msg, e);
     } finally {
       closeClient();
@@ -144,7 +144,7 @@ public class DefaultSentryAccessController extends SentryHiveAccessController {
       HiveOperation hiveOp = HiveOperation.DROPROLE;
       executeOnFailureHooks(hiveOp, e);
     } catch (SentryUserException e) {
-      String msg = "Error occurred when Sentry client creating role: " + e.getMessage();
+      String msg = "Sentry failed to drop role: " + e.getMessage();
       executeOnErrorHooks(msg, e);
     } finally {
       closeClient();

http://git-wip-us.apache.org/repos/asf/sentry/blob/71daaf03/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java
----------------------------------------------------------------------
diff --git a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java
index 0f93b42..14a9619 100644
--- a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java
+++ b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryAccessController.java
@@ -120,7 +120,7 @@ public class DefaultSentryAccessController extends SentryHiveAccessController {
       HiveOperation hiveOp = HiveOperation.CREATEROLE;
       executeOnFailureHooks(hiveOp, e);
     } catch (SentryUserException e) {
-      String msg = "Error occurred when Sentry client creating role: " + e.getMessage();
+      String msg = "Sentry failed to create role: " + e.getMessage();
       executeOnErrorHooks(msg, e);
     } finally {
       closeClient();
@@ -141,7 +141,7 @@ public class DefaultSentryAccessController extends SentryHiveAccessController {
       HiveOperation hiveOp = HiveOperation.DROPROLE;
       executeOnFailureHooks(hiveOp, e);
     } catch (SentryUserException e) {
-      String msg = "Error occurred when Sentry client creating role: " + e.getMessage();
+      String msg = "Sentry failed to drop role: " + e.getMessage();
       executeOnErrorHooks(msg, e);
     } finally {
       closeClient();