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/11/16 09:56:53 UTC
[16/32] sentry git commit: SENTRY-2017: Fix Sentry e2e tests to use
SentryMetastorePostEventListenerNotificationLog. (Kalyan Kumar Kalvagadda,
reviewed-by Sergio Pena and Na Li)
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/akolb-cli
Commit: 8de33e0855f5254595efebe2c2d593eea0fe13b3
Parents: cdd0314
Author: Kalyan Kumar Kalvagadda <kk...@cloudera.com>
Authored: Wed Nov 8 16:58:40 2017 -0600
Committer: Kalyan Kumar Kalvagadda <kk...@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,