SENTRY-1376: Fix alter property case correctly - Deletes ACLS on the table (Sravya Tirukkovalur, Reviewed by: Hao Hao)
Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/218be0c5 Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/218be0c5 Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/218be0c5 Branch: refs/heads/sentry-ha-redesign Commit: 218be0c59efb7d4f57bcb64e8d5fef8c51e06556 Parents: 65568d3 Author: Alexander Kolbasov <[email protected]> Authored: Fri Mar 10 21:50:32 2017 -0800 Committer: Alexander Kolbasov <[email protected]> Committed: Fri Mar 10 21:50:32 2017 -0800 ---------------------------------------------------------------------- .../binding/metastore/SentryMetastorePostEventListener.java | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/218be0c5/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java ---------------------------------------------------------------------- diff --git a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java index 51eebd9..fdb6df4 100644 --- a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java +++ b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java @@ -408,10 +408,8 @@ public class SentryMetastorePostEventListener extends MetaStoreEventListener { } // The HDFS plugin needs to know if it's a path change (set location) for (SentryMetastoreListenerPlugin plugin : sentryPlugins) { - if(!oldPath.equals(newPath)) { - plugin.renameAuthzObject(oldDbName + "." + oldTabName, oldPath, + plugin.renameAuthzObject(oldDbName + "." + oldTabName, oldPath, newDbName + "." + newTabName, newPath); - } } }
