sentry-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From kal...@apache.org
Subject sentry git commit: SENTRY-2017: Fix Sentry e2e tests to use SentryMetastorePostEventListenerNotificationLog. (Kalyan Kumar Kalvagadda, reviewed-by Sergio Pena and Na Li)
Date Wed, 08 Nov 2017 23:00:37 GMT
Repository: sentry
Updated Branches:
  refs/heads/master cdd0314ee -> 8de33e085


SENTRY-2017: Fix Sentry e2e tests to use SentryMetastorePostEventListenerNotificationLog.
(Kalyan Kumar Kalvagadda, reviewed-by Sergio Pena and Na Li)


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

Branch: refs/heads/master
Commit: 8de33e0855f5254595efebe2c2d593eea0fe13b3
Parents: cdd0314
Author: Kalyan Kumar Kalvagadda <kkalyan@cloudera.com>
Authored: Wed Nov 8 16:58:40 2017 -0600
Committer: Kalyan Kumar Kalvagadda <kkalyan@cloudera.com>
Committed: Wed Nov 8 16:58:40 2017 -0600

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


http://git-wip-us.apache.org/repos/asf/sentry/blob/8de33e08/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