> On Nov. 9, 2017, 4:18 p.m., Na Li wrote: > > should we remove the definition of profile hive-authz2 as well?
Not yet. The idea is to keep this profile for a little while until we make sure the transition is completely (without issues). There is still code and tests that could have been forgotten to bring, so better keep it for a while. - Sergio ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63667/#review190600 ----------------------------------------------------------- On Nov. 8, 2017, 3 p.m., Sergio Pena wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/63667/ > ----------------------------------------------------------- > > (Updated Nov. 8, 2017, 3 p.m.) > > > Review request for sentry. > > > Bugs: sentry-2037 > https://issues.apache.org/jira/browse/sentry-2037 > > > Repository: sentry > > > Description > ------- > > Remove the sentry-binding-hive-v2 dependency from the main pom. > > > Diffs > ----- > > pom.xml bea2c5fe4bae53eb6b45602f2586ef30e9e7c309 > > > Diff: https://reviews.apache.org/r/63667/diff/1/ > > > Testing > ------- > > > Thanks, > > Sergio Pena > >