Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-28 Thread Sergio Pena
Kalyan, I cherry-picked HIVE-17048 to all 2.x versions: 2.1.2, 2.2.1,2.3.1 and 2.4.0 They are not released yet, but as soon as one of them is released, then we can bump the Hive version of Sentry to the new version. - Sergio On Fri, Aug 11, 2017 at 12:30 PM, Kalyan Kumar Kalvagadda < kkal...@cl

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-11 Thread Kalyan Kumar Kalvagadda
What I've heard from one of the Hive PMC is that they could get the fix to Hive 2.4. That is the closest. I will follow-up with him again. Colm's suggestion is reasonable. Picking Hive 2.0.1 might be safer as it might have fixes that were found in Hive 2.0.0 -Kalyan On Fri, Aug 11, 2017 at 2:3

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-11 Thread Colm O hEigeartaigh
It's unfortunate, but it seems like option 1 is the only solution really (although we should pick up Hive 2.0.1 at least). Perhaps the Hive community might be receptive to backport the fix to 2.3.x, 2.2.x, 2.1.x given that it is a simple fix and a blocker for Sentry? Colm. On Thu, Aug 10, 2017 at

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-10 Thread Mat Crocker
My opinion (non-binding) is option-1. Keep it simple and follow HIVE. On Thu, Aug 10, 2017 at 5:17 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Lina, > > I don't have that information. I need to dig deeper into apache hive wiki > to get some understanding of it. > > -Kalyan > > -K

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-10 Thread Kalyan Kumar Kalvagadda
Lina, I don't have that information. I need to dig deeper into apache hive wiki to get some understanding of it. -Kalyan -Kalyan On Wed, Aug 9, 2017 at 12:09 PM, Na Li wrote: > What is the timeline of back porting the fix to Hive 2.4? And when do we > plan to release sentry 2.0? > > On Wed, A

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-09 Thread Na Li
What is the timeline of back porting the fix to Hive 2.4? And when do we plan to release sentry 2.0? On Wed, Aug 9, 2017 at 11:57 AM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Hello all, > > I would like to share issue that i have discovered while working on > SENTRY-1838

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-08-09 Thread Kalyan Kumar Kalvagadda
Hello all, I would like to share issue that i have discovered while working on SENTRY-1838 (Support Hive 2.1.1 for sentry 2.0) With the design changes done in Hive between 2.0.0 through 2.1.1, HiveOperation is not passed to HiveSemanticAnalyzerHo

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-19 Thread Kalyan Kumar Kalvagadda
Lina, Apache community is currently focused on sentry-ha to be released. It has highest priority over others. If you are contributing to that, I would say continue doing so. -Kalyan On Wed, Jul 19, 2017 at 10:39 AM, Na Li wrote: > Kalyan, > > I can work on those items. > > Can you create seper

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-19 Thread Na Li
Kalyan, I can work on those items. Can you create seperate Jiras for each one and add more info. In terms of priority, is finishing sentry-HA a higher priority than these (for C6)? Thanks, Lina On Wed, Jul 19, 2017 at 10:02 AM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Hello a

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-19 Thread Kalyan Kumar Kalvagadda
Hello all, Either way we need to fix couple of things first. 1. Fix the test failures when hive-authz2 and datanucleus4 profiles are enabled. 2. Update e2e tests in sentry-tests-hive-v2 package to have same coverage as tests in sentry-tests-hive package. Does any one wants to take ownership of a

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-18 Thread Na Li
I agree with Colm too. We can support Hive 2.1.1 only in sentry 2.0 Lina On Tue, Jul 18, 2017 at 10:03 AM, Sergio Pena wrote: > I agree with Colm too. Hive 1.1 is too old and due to the way how Sentry > works with HMS and notifications, it's not worth the effort to keep > supporting Hive 1.1. L

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-18 Thread Sergio Pena
I agree with Colm too. Hive 1.1 is too old and due to the way how Sentry works with HMS and notifications, it's not worth the effort to keep supporting Hive 1.1. Let's drop it. On Tue, Jul 18, 2017 at 8:47 AM, Colm O hEigeartaigh wrote: > I think we should drop support for Hive 1.1.0 in sentry 2

Re: Sentry 2.0.0 depends on multiple versions of Hive

2017-07-18 Thread Colm O hEigeartaigh
I think we should drop support for Hive 1.1.0 in sentry 2.0.0. Hive 1.1.0 users can obtain the plugin from the Sentry 1.8.x series of releases. Colm. On Tue, Jul 18, 2017 at 1:18 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Hello all, > > Sentry currently supports Hive1.1.0 and H

Sentry 2.0.0 depends on multiple versions of Hive

2017-07-18 Thread Kalyan Kumar Kalvagadda
Hello all, Sentry currently supports Hive1.1.0 and Hive 2.0.0 based on the profile used. I'm currently working on sentry 2.0.0 release upstream to integrate with Hive 2.1.1. I'm sending this mail seeking inputs on what is the support we need in Sentry 2.0.0 release. Would that be just Hive 2.x or