----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63974/#review191686 -----------------------------------------------------------
Code looks good overall. I have some questions/clarifications which I will be posting tomorrow morning. - kalyan kumar kalvagadda On Nov. 21, 2017, 10:14 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > > 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/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >