sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From vam...@apache.org
Subject [36/52] [abbrv] sentry git commit: SENTRY-1783: alterSentryRoleGrantPrivilegeCore does more persistence work than required (Alex Kolbasov, reviewed by: Kalyan Kalvagadda, Na Li, Vamsee Yarlagadda)
Date Wed, 14 Jun 2017 00:57:14 GMT
SENTRY-1783: alterSentryRoleGrantPrivilegeCore does more persistence work than required (Alex
Kolbasov, reviewed by: Kalyan Kalvagadda, Na Li, Vamsee Yarlagadda)

Change-Id: Ic404c323d5809aa10894f6628ce8f6756a6bc001
Reviewed-on: http://gerrit.sjc.cloudera.com:8080/23120
Tested-by: Jenkins User
Reviewed-by: Alexander Kolbasov <akolb@cloudera.com>


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

Branch: refs/for/cdh5-1.5.1_ha
Commit: d31a48705a5ec24de3f46de86d1dbea0d645977e
Parents: 6c2fb96
Author: Alexander Kolbasov <akolb@cloudera.com>
Authored: Fri May 26 14:53:11 2017 -0700
Committer: Alexander Kolbasov <akolb@cloudera.com>
Committed: Fri May 26 16:09:33 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/d31a4870/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 26db845..059cf20 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
@@ -672,7 +672,6 @@ public class SentryStore {
         mPrivilege = convertToMSentryPrivilege(privilege);
       }
       mPrivilege.appendRole(mRole);
-      pm.makePersistent(mRole);
       pm.makePersistent(mPrivilege);
     }
     return mPrivilege;


Mime
View raw message