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 <[email protected]> Authored: Mon Oct 23 15:37:10 2017 -0500 Committer: Kalyan Kumar Kalvagadda <[email protected]> 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,
