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 <[email protected]> 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 <[email protected]> Authored: Fri May 26 14:53:11 2017 -0700 Committer: Alexander Kolbasov <[email protected]> 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;
