-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42364/
-----------------------------------------------------------
(Updated Jan. 19, 2016, 10:21 p.m.)
Review request for sentry and Hao Hao.
Changes
-------
On a second thought, removed the code to catch exceptions in clean up codes.
Bugs: SENTRY-1009
https://issues.apache.org/jira/browse/SENTRY-1009
Repository: sentry
Description
-------
We started to see many TestDatabaseProvider test failures because it validates
number of test objects instead of test object names. Because there are some
system roles created in the environment, so the number of objects are usually
larger than expected. Improved the test cases. Also catch the exception
whenever cleanup fails since we don't need to do anything.
Diffs (updated)
-----
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDatabaseProvider.java
98de57d3d749c3f887632ff1461ef05dd29376c1
Diff: https://reviews.apache.org/r/42364/diff/
Testing
-------
mvn test -Dtest=TestDatabaseProvider passed locally testing enviroments.
Thanks,
Anne Yu
