Re: Review Request 66792: SENTRY-1949: Old full snapshots are never cleaned up

2018-04-26 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66792/ --- (Updated April 27, 2018, 3:57 a.m.) Review request for sentry, Alexander

Review Request 66837: SENTRY-2170: Update the Sentry-HDFS thrift for user level privileges.

2018-04-26 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66837/ --- Review request for sentry, Na Li and Sergio Pena. Bugs: SENTRY-2170

Re: Review Request 66826: SENTRY-2213: Increase schema version from 2.0.0 to 2.1.0

2018-04-26 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66826/#review202028 --- Ship it! Ship It! - Sergio Pena On April 26, 2018, 7:22

Re: Review Request 66826: SENTRY-2213: Increase schema version from 2.0.0 to 2.1.0

2018-04-26 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66826/#review202027 --- Ship it! Ship It! - Arjun Mishra On April 26, 2018, 7:22

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

2018-04-26 Thread Steve Moist via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66748/ --- (Updated April 26, 2018, 7:38 p.m.) Review request for sentry. Repository:

Re: Sentry Release Schedule for FGP and ABAC

2018-04-26 Thread Sergio Pena
Hey Stephen, I'm not totally against having a feature branch, but we had a bad experience with the Sentry HA re-design where all the work was done in a feature branch. After doing so many fixes and improvements in that sentry-ha branch, it was difficult to merge it into master, so we decided to

Review Request 66826: SENTRY-2213: Increase schema version from 2.0.0 to 2.1.0

2018-04-26 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66826/ --- Review request for sentry, Alexander Kolbasov, Arjun Mishra, kalyan kumar

Re: HMS catalogs and Sentry

2018-04-26 Thread Kalyan Kumar Kalvagadda
I feel, we need to understand it better before considering to put in sentry wish list. *Thanks,Kalyan Kumar Kalvagadda* | Software Engineer t. (469) 279- <00>5732 cloudera.com [image: Cloudera] [image: Cloudera on Twitter]

Re: Sentry Release Schedule for FGP and ABAC

2018-04-26 Thread Na Li
Stephen, I think it is OK for two features to be developed in parallel at master as long as the new updates do not break existing features. If ABAC feature is not complete when we release sentry 2.1, can you turn off ABAC in master branch, so it does not cause the release with incomplete

Re: HMS catalogs and Sentry

2018-04-26 Thread Na Li
Sasha, The jira https://issues.apache.org/jira/browse/HIVE-18755 has no description on what's the motivation of having catalog and how it works. Is catalog similar to namespace, so all databases in a catalog can be managed by one entity? Do you know any description that would help us

Sentry Release Schedule for FGP and ABAC

2018-04-26 Thread Stephen Moist
Hey all, what does the current roadmap and release schedule look like for FGP and ABAC? I’ve been told that FGP is going out in the next release, ABAC is more slated for the summer. How do we want to handle simultaneous development of these features? For ABAC, our dev process is more agile.

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

2018-04-26 Thread Steve Moist via Review Board
> On April 25, 2018, 4:51 p.m., Colm O hEigeartaigh wrote: > > Why is the package name changed for GenericPrivilegeConverter.java + > > TSentryPrivilegeConverter.java? They've gone from > > "sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools" > > to

Re: Review Request 66764: SENTRY-2210: AUTHZ_PATH should have index on the foreign key AUTHZ_OBJ_ID

2018-04-26 Thread Sergio Pena via Review Board
> On April 25, 2018, 8:21 p.m., Sergio Pena wrote: > > Lina, would it be better to create another patch to fix the 2.1.0 versions? > > So we can have this patch to add the index only instead of several new > > files, what do you think? > > Na Li wrote: > Sergio, I don't want to take the

Re: Review Request 66748: SENTRY-2207 Refactor out Sentry CLI from sentry-provider-db into own module

2018-04-26 Thread Colm O hEigeartaigh
> On April 25, 2018, 4:51 p.m., Colm O hEigeartaigh wrote: > > Why is the package name changed for GenericPrivilegeConverter.java + > > TSentryPrivilegeConverter.java? They've gone from > > "sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/tools" > > to