sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ak...@apache.org
Subject sentry git commit: SENTRY-1783: alterSentryRoleGrantPrivilegeCore does more persistence work than required (Alex Kolbasov, reviewed by: Kalyan Kalvagadda, Na Li, Vamsee Yarlagadda)
Date Fri, 26 May 2017 06:25:08 GMT
Repository: sentry
Updated Branches:
  refs/heads/sentry-ha-redesign 75a8faf15 -> 9c11c32c6


SENTRY-1783: alterSentryRoleGrantPrivilegeCore does more persistence work than required (Alex
Kolbasov, reviewed by: Kalyan Kalvagadda, Na Li, Vamsee Yarlagadda)


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

Branch: refs/heads/sentry-ha-redesign
Commit: 9c11c32c6a571d710c9317f61679d0de99ff7d04
Parents: 75a8faf
Author: Alexander Kolbasov <akolb@cloudera.com>
Authored: Thu May 25 23:24:47 2017 -0700
Committer: Alexander Kolbasov <akolb@cloudera.com>
Committed: Thu May 25 23:24:47 2017 -0700

----------------------------------------------------------------------
 .../apache/sentry/provider/db/service/persistent/SentryStore.java   | 1 -
 1 file changed, 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/9c11c32c/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
----------------------------------------------------------------------
diff --git a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
index 8450402..f26894a 100644
--- a/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
+++ b/sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java
@@ -696,7 +696,6 @@ public class SentryStore {
         mPrivilege = convertToMSentryPrivilege(privilege);
       }
       mPrivilege.appendRole(mRole);
-      pm.makePersistent(mRole);
       pm.makePersistent(mPrivilege);
     }
     return mPrivilege;


Mime
View raw message