----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47267/#review132765 -----------------------------------------------------------
Ship it! Ship It! - Sravya Tirukkovalur On May 11, 2016, 10:26 p.m., Ashish Singh wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47267/ > ----------------------------------------------------------- > > (Updated May 11, 2016, 10:26 p.m.) > > > Review request for sentry, Colin Ma, Dapeng Sun, Hao Hao, and Sravya > Tirukkovalur. > > > Bugs: SENTRY-1253 > https://issues.apache.org/jira/browse/SENTRY-1253 > > > Repository: sentry > > > Description > ------- > > SentryShellKafka is incorrectly setting component as "KAFKA" > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools/SentryShellKafka.java > e15d8d298b0d291d159afcba978a70c48d7f23a8 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/tools/TestSentryShellKafka.java > d49bc57b5c0b619e5e0e34fcf1f2676535c9e13c > > Diff: https://reviews.apache.org/r/47267/diff/ > > > Testing > ------- > > Ran end-to-end tests and unit tests. Tested by deploying bits on a real > cluster. > > > Thanks, > > Ashish Singh > >