-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63974/
-----------------------------------------------------------
(Updated Nov. 21, 2017, 10:14 p.m.)
Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and Na
Li.
Changes
-------
Rebased with latest master changes.
Bugs: sentry-2062
https://issues.apache.org/jira/browse/sentry-2062
Repository: sentry
Description
-------
Switch unit-tests to use the Hive DbNotificationListener and refactor all the
SentryJSON* messages to implement the HMS API instead of the HCAT API.
Diffs (updated)
-----
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAddPartitionMessage.java
10d7a934ae27935fc446ce44cd6ade15bd527e4b
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterPartitionMessage.java
25de8083a9e62f70de638f7f35aba6e45ed6ec77
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONAlterTableMessage.java
4670494a1522a3fa61323159d0e0ddd5ee840f78
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONCreateDatabaseMessage.java
8c627586ade1f7deeeceea47a2cea6023a90f761
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONCreateTableMessage.java
d15bc48886107fe1bb944642a31704408d13cf23
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONDropDatabaseMessage.java
c8e7c75c8f99a7af3776519544a039f880a476db
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONDropPartitionMessage.java
d5f899c8e6948f73d2e9beb6e3552c83954d58b8
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONDropTableMessage.java
e67f56243d0b08a9c9892e7a2646c44d4e58d936
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONMessageDeserializer.java
929ac8c05f4d24044139b26b8e6bc6dc5791cc81
sentry-binding/sentry-binding-hive-follower/src/main/java/org/apache/sentry/binding/metastore/messaging/json/SentryJSONMessageFactory.java
0af02d1c164f482f90a839eff66f95be6ad49d55
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListener.java
11b6b4add0643999b491cd8c6e09b4efa50c10ed
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListenerBase.java
40cf17a470c41ac81218a0472da368aa502fac65
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/metastore/SentryMetastorePostEventListenerNotificationLog.java
90502310a77d7dc6ebbeb9b1d5783c05fc2b6063
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/FullUpdateModifier.java
25019708411c66c43700f7219b4059dd31af80a3
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/NotificationProcessor.java
d92f23e637769330775d00152c363529572186d1
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryHMSClient.java
6ec163b1d802810287a55bc5cd98409bc73e8ba8
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/persistent/TestHMSFollowerSentryStoreIntegration.java
928d0c8d861d4414bb53bf41e1c0353332381654
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestFullUpdateModifier.java
4f90c3e68f0c76e08fb590bce26e560f2b7ae1ed
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestHMSFollower.java
5dd9b1bdbfc75a6606c6f0f600427f45189d4bed
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/TestNotificationProcessor.java
00f9b3959b7ccf9d046c422a007ff375eddba076
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java
d9aec8b6d6c01ff179b369e34b8f768a2340ce20
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
708733e0a5f27dce27c03986c7fe4632993dde8e
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestDBNotificationListenerInBuiltDeserializer.java
ea44e33f17f4e226fb0af1366d2aa6d65115f5d9
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/metastore/TestDbNotificationListenerSentryDeserializer.java
8e588b1d954d4ed90beb464055b2fcc07c7e72c1
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
3a404dd446932e5ceb1e37fc4315d506d0e1e1fd
Diff: https://reviews.apache.org/r/63974/diff/2/
Changes: https://reviews.apache.org/r/63974/diff/1-2/
Testing
-------
Thanks,
Sergio Pena
