-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/50611/
-----------------------------------------------------------
(Updated Aug. 16, 2016, 6:17 p.m.)
Review request for sentry, Hao Hao and Sravya Tirukkovalur.
Changes
-------
Addressed previous comments;
Also updated a couple of files to test sentry ha and pool based clients.
Bugs: SENTRY-1423
https://issues.apache.org/jira/browse/SENTRY-1423
Repository: sentry
Description
-------
Sentry HA Test: test programmatic failover, validate privileges before and
afterwards. validate expected exception is thrown.
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryServiceClientFactory.java
56d774b883914dbffff438101f47fb94518c94d5
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/ha/TestProgrammaticFailover.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/ha/TestSentryHABase.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
7dc3d0f30583b9edd16015218b199e298d192e8c
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java
3f57a003903961a6aea98bd583a14b65bd2b98a2
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java
054b1935975368db058791102f3892f027e31636
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java
dac11517f36a743dff8730465fbf2397ebf16c88
Diff: https://reviews.apache.org/r/50611/diff/
Testing
-------
Thanks,
Anne Yu
