You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by br...@apache.org on 2014/02/25 03:52:46 UTC
[03/26] git commit: Revert commit of SENTRY-78
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/db_policy_store
Commit: 319d64a4a31182dd73a7368d69fd1924f07bb968
Parents: 4baffe9
Author: Shreepadma Venugopalan <sh...@apache.org>
Authored: Thu Dec 26 16:42:08 2013 -0800
Committer: Shreepadma Venugopalan <sh...@apache.org>
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