----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45329/#review128053 -----------------------------------------------------------
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryShellKafka.java (line 33) <https://reviews.apache.org/r/45329/#comment191445> Does it mean that the end user will not use SentryShellKafka? - Hao Hao On March 25, 2016, 1:13 a.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45329/ > ----------------------------------------------------------- > > (Updated March 25, 2016, 1:13 a.m.) > > > Review request for sentry, Dapeng Sun and Hao Hao. > > > Bugs: SENTRY-1162 > https://issues.apache.org/jira/browse/SENTRY-1162 > > > Repository: sentry > > > Description > ------- > > SENTRY-1162: Add shell for Sentry Kafka integration > > > Diffs > ----- > > sentry-provider/sentry-provider-db/pom.xml > bf4dfdc1de90b1018767e2a61bee970655f02682 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/KafkaTSentryPrivilegeConvertor.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryShellKafka.java > PRE-CREATION > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryShellKafka.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/45329/diff/ > > > Testing > ------- > > Added tests. > > > Thanks, > > Ashish Singh > >
