sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cohei...@apache.org
Subject [4/6] sentry git commit: SENTRY-2017: Fix Sentry e2e tests to use SentryMetastorePostEventListenerNotificationLog.
Date Fri, 27 Oct 2017 09:22:25 GMT
SENTRY-2017: Fix Sentry e2e tests to use SentryMetastorePostEventListenerNotificationLog.


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

Branch: refs/heads/akolb-cli
Commit: 99efc7fc960d5680fc7605400e0375a7883bae37
Parents: cfa8abb
Author: Kalyan Kumar Kalvagadda <kkalyan@cloudera.com>
Authored: Mon Oct 23 15:37:10 2017 -0500
Committer: Kalyan Kumar Kalvagadda <kkalyan@cloudera.com>
Committed: Thu Oct 26 13:56:29 2017 -0500

----------------------------------------------------------------------
 .../tests/e2e/hive/AbstractTestWithStaticConfiguration.java   | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/sentry/blob/99efc7fc/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
----------------------------------------------------------------------
diff --git a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
index 5fb4659..630bc68 100644
--- a/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
+++ b/sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
@@ -128,7 +128,6 @@ public abstract class AbstractTestWithStaticConfiguration extends RulesForE2ETes
   protected static final String SERVER_HOST = "localhost";
   private static final String EXTERNAL_SENTRY_SERVICE = "sentry.e2etest.external.sentry";
   protected static final String EXTERNAL_HIVE_LIB = "sentry.e2etest.hive.lib";
-  private static final String ENABLE_NOTIFICATION_LOG = "sentry.e2etest.enable.notification.log";
 
   protected static boolean policyOnHdfs = false;
   protected static boolean defaultFSOnHdfs = false;
@@ -152,7 +151,7 @@ public abstract class AbstractTestWithStaticConfiguration extends RulesForE2ETes
   protected static Map<String, String> properties;
   protected static SentrySrv sentryServer;
   protected static Configuration sentryConf;
-  protected static boolean enableNotificationLog = false;
+  protected static boolean enableNotificationLog = true;
   protected static Context context;
   protected final String semanticException = "SemanticException No valid privileges";
 
@@ -282,10 +281,6 @@ public abstract class AbstractTestWithStaticConfiguration extends RulesForE2ETes
       policyURI = policyFileLocation.getPath();
     }
 
-    if ("true".equalsIgnoreCase(System.getProperty(ENABLE_NOTIFICATION_LOG, "false"))) {
-      enableNotificationLog = true;
-    }
-
     if (enableHiveConcurrency) {
       properties.put(HiveConf.ConfVars.HIVE_SUPPORT_CONCURRENCY.varname, "true");
       properties.put(HiveConf.ConfVars.HIVE_TXN_MANAGER.varname,


Mime
View raw message