----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29347/#review67360 -----------------------------------------------------------
Ship it! sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java <https://reviews.apache.org/r/29347/#comment111372> Looks like some tabs got added, can you remove those? sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java <https://reviews.apache.org/r/29347/#comment111373> perhaps rename to runTestAsSubject? - Lenni Kuff On Dec. 25, 2014, 10:50 a.m., Dapeng Sun wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/29347/ > ----------------------------------------------------------- > > (Updated Dec. 25, 2014, 10:50 a.m.) > > > Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, shen > guoquan, Lenni Kuff, Prasad Mujumdar, and Sravya Tirukkovalur. > > > Bugs: SENTRY-595 > https://issues.apache.org/jira/browse/SENTRY-595 > > > Repository: sentry > > > Description > ------- > > Working with Colin Ma on SENTRY-296, we found client will throw a Kerberos > exception when do some operations like renew client in Kerberos related > UnitTests. The root cause is in some test cases, client didn't run under > clientSubject > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java > 4a6cac9 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java > be14afd > > Diff: https://reviews.apache.org/r/29347/diff/ > > > Testing > ------- > > Unit Tests in local > > > Thanks, > > Dapeng Sun > >
