Updated Branches: refs/heads/master 4baffe9b4 -> 319d64a4a
Revert commit of SENTRY-78 Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/319d64a4 Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/319d64a4 Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/319d64a4 Branch: refs/heads/master Commit: 319d64a4a31182dd73a7368d69fd1924f07bb968 Parents: 4baffe9 Author: Shreepadma Venugopalan <[email protected]> Authored: Thu Dec 26 16:42:08 2013 -0800 Committer: Shreepadma Venugopalan <[email protected]> Committed: Thu Dec 26 16:42:08 2013 -0800 ---------------------------------------------------------------------- .../apache/sentry/binding/hive/HiveAuthzBindingHook.java | 9 --------- .../tests/e2e/hive/TestPrivilegesAtDatabaseScope.java | 4 ---- 2 files changed, 13 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/319d64a4/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java ---------------------------------------------------------------------- diff --git a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java index 0dd28b7..7f9560f 100644 --- a/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java +++ b/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/HiveAuthzBindingHook.java @@ -370,15 +370,6 @@ implements HiveDriverFilterHook { } for(ReadEntity readEntity:inputs) { - // If this is a UDF, then check whether its allowed to be executed - // TODO: when we support execute privileges on UDF, this can be removed. - if (isUDF(readEntity)) { - if (isBuiltinUDF(readEntity)) { - checkUDFWhiteList(readEntity.getUDF().getDisplayName()); - } - continue; - } - List<DBModelAuthorizable> entityHierarchy = new ArrayList<DBModelAuthorizable>(); entityHierarchy.add(hiveAuthzBinding.getAuthServer()); entityHierarchy.addAll(getAuthzHierarchyFromEntity(readEntity)); http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/319d64a4/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java ---------------------------------------------------------------------- diff --git a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java index 8c145ca..82d73e5 100644 --- a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java +++ b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPrivilegesAtDatabaseScope.java @@ -107,10 +107,6 @@ public class TestPrivilegesAtDatabaseScope extends AbstractTestWithStaticConfigu statement.execute("CREATE TABLE DB_1.TAB_2(A STRING)"); statement.execute("LOAD DATA LOCAL INPATH '" + dataFile.getPath() + "' INTO TABLE DB_1.TAB_2"); - // test CTAS can reference UDFs - statement.execute("USE DB_1"); - statement.execute("create table table2 as select A, count(A) from TAB_1 GROUP BY A"); - // test user can switch db statement.execute("USE DB_1"); //test user can create view
