----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/22314/#review45038 -----------------------------------------------------------
Ship it! LGTM - Prasad Mujumdar On June 6, 2014, 7:31 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22314/ > ----------------------------------------------------------- > > (Updated June 6, 2014, 7:31 p.m.) > > > Review request for sentry. > > > Bugs: SENTRY-266 > https://issues.apache.org/jira/browse/SENTRY-266 > > > Repository: sentry > > > Description > ------- > > I've added call to Hadoop API that will do the resolution the same way as is > done in the rest of the platform. > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/thrift/SentryPolicyServiceClient.java > 62113c8 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java > eefcb0a > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceWithKerberos.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/service/thrift/SentryServiceIntegrationBase.java > f3142fe > > Diff: https://reviews.apache.org/r/22314/diff/ > > > Testing > ------- > > Added test case that exercise the server principal with _HOST. > > > Thanks, > > Jarek Cecho > >
