Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-19 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/#review180910 --- Ship it! Ship It! - Colm O hEigeartaigh On July 18, 2017, 11

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-18 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/#review180884 --- Ship it! Ship It! - Sergio Pena On July 18, 2017, 11:12 p.m.

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-18 Thread kalyan kumar kalvagadda
> On July 17, 2017, 8:11 p.m., Colm O hEigeartaigh wrote: > > > RELEASE > > > > Is this a mistake? > > kalyan kumar kalvagadda wrote: > no it is not. Currently when hive-authz1 profiles is used sentry depends > on Hive 1.1.0 for all the hive dependencies except for > hive-hcatalog-server-

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-18 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/ --- (Updated July 18, 2017, 11:12 p.m.) Review request for sentry, Colm O hEigearta

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-17 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/#review180753 --- pom.xml Lines 76 (patched)

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-17 Thread kalyan kumar kalvagadda
> On July 17, 2017, 8:11 p.m., Colm O hEigeartaigh wrote: > > > RELEASE > > > > Is this a mistake? > > kalyan kumar kalvagadda wrote: > no it is not. Currently when hive-authz1 profiles is used sentry depends > on Hive 1.1.0 for all the hive dependencies except for > hive-hcatalog-server-

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-17 Thread kalyan kumar kalvagadda
> On July 17, 2017, 8:11 p.m., Colm O hEigeartaigh wrote: > > > RELEASE > > > > Is this a mistake? no it is not. Currently when hive-authz1 profiles is used sentry depends on Hive 1.1.0 for all the hive dependencies except for hive-hcatalog-server-extensions. It uses release version to get th

Re: Review Request 60904: SENTRY-1849 Fix the pom file to use appropriate hive dependencies based on hive-authz profile

2017-07-17 Thread Colm O hEigeartaigh
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60904/#review180729 --- > RELEASE Is this a mistake? - Colm O hEigeartaigh On July 17,