Repository: sentry Updated Branches: refs/heads/master c6d139762 -> 260d11abf
SENTRY-2312: Update owner privileges for table when owner is changed. (Kalyan Kumar Kalvagadda reviewed by Lina li) Project: http://git-wip-us.apache.org/repos/asf/sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/sentry/commit/260d11ab Tree: http://git-wip-us.apache.org/repos/asf/sentry/tree/260d11ab Diff: http://git-wip-us.apache.org/repos/asf/sentry/diff/260d11ab Branch: refs/heads/master Commit: 260d11abf85fa3f36952c05dae82ba706269e297 Parents: c6d1397 Author: Kalyan Kumar Kalvagadda <[email protected]> Authored: Fri Aug 17 11:35:54 2018 -0500 Committer: Kalyan Kumar Kalvagadda <[email protected]> Committed: Fri Aug 17 11:35:54 2018 -0500 ---------------------------------------------------------------------- .../org/apache/sentry/binding/metastore/SentryHmsEvent.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/sentry/blob/260d11ab/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryHmsEvent.java ---------------------------------------------------------------------- diff --git a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryHmsEvent.java b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryHmsEvent.java index e5d4437..e0f35e8 100644 --- a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryHmsEvent.java +++ b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryHmsEvent.java @@ -181,9 +181,9 @@ class SentryHmsEvent { private void setOwnerInfo(Table table) { ownerName = (table != null) ? table.getOwner() : null; - // Hive 2.3.2 currently support owner type. Assuming user as the type for now. - // TODO once sentry dependency is changed to a hive version that suppots user type for table this - // hard coding should be rempved. + // TODO once sentry dependency is changed to a hive version that has fix for HIVE-18762 + // below hard coding should be removed. + // ownerType = mapOwnerType.get(table.getOwnerType()); ownerType = TSentryPrincipalType.USER; }
