----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52795/ -----------------------------------------------------------
(Updated Jan. 10, 2017, 12:01 a.m.) Review request for sentry, Alexander Kolbasov, Anne Yu, kalyan kumar kalvagadda, Li Li, and Sravya Tirukkovalur. Repository: sentry Description ------- SENTRY-1499: Add feature flag for using NotificationLog 1. Keep two SentryMetastorePostEventListeners. One for using notificationlog, another for not(Used the original SentryMetastorePostEventListeners from master https://github.com/apache/sentry/blob/master/sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java). 2. Add feature flag for using HMSFollower and SentryMetastorePostEventListeners with notification log. Change-Id: I46a11d27ae0159a735b6049e9b7c78216d0e5346 Diffs (updated) ----- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java 75190c1805162e11baf7c4553668693e4e6bb3c4 sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListenerNotificationLog.java PRE-CREATION sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java bd4ada44bf882214dc20255bf902a0fbf1a0393c sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java c4fdf1d25e9b669b4df1fde69961fd33f08df732 sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java 61b24fa0e9f221b75ae343fc534b4d82fbce272a sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestSentryListenerInBuiltDeserializer.java c4be62db71e67099f22f3ba8991e03a32fb0f28a sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestSentryListenerSentryDeserializer.java 6f1886f942ca80ab4c356b81777d3ac336017292 Diff: https://reviews.apache.org/r/52795/diff/ Testing ------- Thanks, Hao Hao