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 <[email protected]> Authored: Wed Nov 8 16:58:40 2017 -0600 Committer: Kalyan Kumar Kalvagadda <[email protected]> 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,
