----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63964/#review191545 -----------------------------------------------------------
sentry-dist/pom.xml Lines 105-108 (original), 105-108 (patched) <https://reviews.apache.org/r/63964/#comment269386> Can you move this dependency out of the hive-authz1 and remove this hive-authz1 profile too? We're not using hive-authz1 anymore. - Sergio Pena On Nov. 20, 2017, 9:48 p.m., Steve Moist wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63964/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2017, 9:48 p.m.) > > > Review request for sentry. > > > Bugs: SENTRY-2049 > https://issues.apache.org/jira/browse/SENTRY-2049 > > > Repository: sentry > > > Description > ------- > > Remove hive-authz2 profile from the sentry-dist module > > > Diffs > ----- > > sentry-dist/pom.xml 69f4fcca > > > Diff: https://reviews.apache.org/r/63964/diff/1/ > > > Testing > ------- > > unit tests ran. > > > Thanks, > > Steve Moist > >