[SENTRY-1879] Sentry e2e tests are trying to test without notification log

• Type: Sub-task
• Status: Resolved
• Priority: Major
• Resolution: Won't Fix
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: Sentry
• Assignee: Na Li
• Reporter: Alex Kolbasov
• Created: 11/Aug/17 01:38
• Updated: 24/Aug/17 02:45
• Resolved: 24/Aug/17 02:45

Description

Looking at AbstractTestWithStaticConfiguration we see that it does the following:

        if (enableNotificationLog) {
          properties.put(HiveConf.ConfVars.METASTORE_EVENT_LISTENERS.varname,
              SentryMetastorePostEventListenerNotificationLog.class.getName());
        } else {
          properties.put(HiveConf.ConfVars.METASTORE_EVENT_LISTENERS.varname,
              SentryMetastorePostEventListener.class.getName());
        }
        properties.put("hcatalog.message.factory.impl.json",
            "org.apache.sentry.binding.metastore.messaging.json.SentryJSONMessageFactory");
      }

Note that we now only support notification log and we don't support HMS without notification log. This and other places should be changed to always use notification log and configure it properly.