You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sentry.apache.org by co...@apache.org on 2017/10/27 09:22:25 UTC
[4/6] sentry git commit: SENTRY-2017: Fix Sentry e2e tests to use
SentryMetastorePostEventListenerNotificationLog.
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 <kk...@cloudera.com>
Authored: Mon Oct 23 15:37:10 2017 -0500
Committer: Kalyan Kumar Kalvagadda <kk...@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,