----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/50611/#review146061 -----------------------------------------------------------
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/ha/TestSentryHABase.java (line 200) <https://reviews.apache.org/r/50611/#comment212439> >>One qq here, in mini-cluster testing environment (internal server), does it also use keytab and kerberos file? In mini-cluster, I think there is no kerberos at e2e. In some UT, we use keytab to login and get the credential. Do you mean the exception is used for avoiding the method is called at external SentryService case? - Dapeng Sun On 八月 17, 2016, 2:17 a.m., Anne Yu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/50611/ > ----------------------------------------------------------- > > (Updated 八月 17, 2016, 2:17 a.m.) > > > Review request for sentry, Hao Hao and Sravya Tirukkovalur. > > > 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 > ----- > > > 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 > >