You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by pr...@apache.org on 2014/06/11 07:49:28 UTC

git commit: SENTRY-288: Dissable MetastoreBinding for test cases that do not require it (Arun Suresh via Prasad Mujumdar)

Repository: incubator-sentry
Updated Branches:
  refs/heads/master ef624165b -> 398183228


SENTRY-288: Dissable MetastoreBinding for test cases that do not require it (Arun Suresh via Prasad Mujumdar)


Project: http://git-wip-us.apache.org/repos/asf/incubator-sentry/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-sentry/commit/39818322
Tree: http://git-wip-us.apache.org/repos/asf/incubator-sentry/tree/39818322
Diff: http://git-wip-us.apache.org/repos/asf/incubator-sentry/diff/39818322

Branch: refs/heads/master
Commit: 39818322830827cc1ddf2ffef22a456688e4c40d
Parents: ef62416
Author: Prasad Mujumdar <pr...@cloudera.com>
Authored: Tue Jun 10 22:47:53 2014 -0700
Committer: Prasad Mujumdar <pr...@cloudera.com>
Committed: Tue Jun 10 22:47:53 2014 -0700

----------------------------------------------------------------------
 .../sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java  | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-sentry/blob/39818322/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java
----------------------------------------------------------------------
diff --git a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java
index 39162df..c0dc939 100644
--- a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java
+++ b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/hiveserver/HiveServerFactory.java
@@ -141,12 +141,12 @@ public class HiveServerFactory {
       if (HiveServer2Type.InternalMetastore.equals(type)) {
         properties.put(METASTORE_URI,
           "thrift://localhost:" + String.valueOf(findPort()));
+        if (!properties.containsKey(METASTORE_HOOK)) {
+          properties.put(METASTORE_HOOK,
+              "org.apache.sentry.binding.metastore.MetastoreAuthzBinding");
+        }
       }
     }
-    if (!properties.containsKey(METASTORE_HOOK)) {
-      properties.put(METASTORE_HOOK,
-          "org.apache.sentry.binding.metastore.MetastoreAuthzBinding");
-    }
     if (!properties.containsKey(METASTORE_BYPASS)) {
       properties.put(METASTORE_BYPASS,
           "hive,impala," + System.getProperty("user.name", ""));