Re: Review Request 63667: SENTRY-2037: Remove not needed sentry-binding-hive-v2 dependency from the main pom.xml

2017-11-09 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63667/#review190651 --- Ship it! Ship It! - Alexander Kolbasov On Nov. 8, 2017, 3 p.

Re: Review Request 63667: SENTRY-2037: Remove not needed sentry-binding-hive-v2 dependency from the main pom.xml

2017-11-09 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63667/#review190614 --- Ship it! Ship It! - Na Li On Nov. 8, 2017, 3 p.m., Sergio Pe

Re: Review Request 63667: SENTRY-2037: Remove not needed sentry-binding-hive-v2 dependency from the main pom.xml

2017-11-09 Thread Sergio Pena via Review Board
> 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 forgo

Re: Review Request 63667: SENTRY-2037: Remove not needed sentry-binding-hive-v2 dependency from the main pom.xml

2017-11-09 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63667/#review190600 --- should we remove the definition of profile hive-authz2 as well? -

Review Request 63667: SENTRY-2037: Remove not needed sentry-binding-hive-v2 dependency from the main pom.xml

2017-11-08 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63667/ --- Review request for sentry. Bugs: sentry-2037 https://issues.apache.org/jira