-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63958/
-----------------------------------------------------------
(Updated Jan. 11, 2018, 7:44 p.m.)
Review request for sentry, Na Li and Sergio Pena.
Changes
-------
Addressed review comments from sergio.
Bugs: SENTRY-2034
https://issues.apache.org/jira/browse/SENTRY-2034
Repository: sentry
Description
-------
Currently, there are no e2e tests that test the functionality of pulling the
notifications from HMS and processing them. Which include updating the update
the permissions based on the HMS updates.
Diffs (updated)
-----
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbPrivilegeCleanupOnDrop.java
21383046f8a5b7386cfe9f6f62c0119542a73382
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessing.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessingBase.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessingWithOutHdfsSync.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessingWithOutSyncOnCreate.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessingWithOutSyncOnDrop.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationAdvanced.java
33ace57134b20189aa10bee48f6aae765a1a1830
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationBase.java
c8eef09ed61e11583838a3dfd3117a9478df23ba
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationEnd2End.java
a909b913e9d2342a7f76a886d3fdabdb1e309b9f
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hdfs/TestHDFSIntegrationWithHA.java
cc0449b1d1d0291049cb1fafebb6d752dd426fbc
Diff: https://reviews.apache.org/r/63958/diff/4/
Changes: https://reviews.apache.org/r/63958/diff/3-4/
Testing
-------
Made sure that new tests added and all the older tests pass.
Thanks,
kalyan kumar kalvagadda
