Re: Review Request 69351: SENTRY-2458: Split web service from server service modules

2018-11-30 Thread Brian Towles via Review Board
> On Nov. 30, 2018, 11:30 a.m., Na Li wrote: > > sentry-dist/src/license/THIRD-PARTY.properties > > Line 34 (original) > > > > > > is this change intentially? No this is an artifact of the dist license thing. This n

Re: Review Request 69415: SENTRY-2463: Revoking ALL or * should revoke any other privilege on the entity

2018-11-30 Thread Sergio Pena via Review Board
> On Nov. 30, 2018, 5:46 p.m., kalyan kumar kalvagadda wrote: > > I understand there is a gap here but we should be carefull in changing the > > current behavior.I assume, the idea here is to find a way to remove all the > > privileges granted to an authrozable(server/database/table) > > > > U

Re: Review Request 69415: SENTRY-2463: Revoking ALL or * should revoke any other privilege on the entity

2018-11-30 Thread Arjun Mishra via Review Board
> On Nov. 30, 2018, 5:46 p.m., kalyan kumar kalvagadda wrote: > > I understand there is a gap here but we should be carefull in changing the > > current behavior.I assume, the idea here is to find a way to remove all the > > privileges granted to an authrozable(server/database/table) > > > > U

Re: Review Request 69415: SENTRY-2463: Revoking ALL or * should revoke any other privilege on the entity

2018-11-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69415/#review210983 --- I understand there is a gap here but we should be carefull in chan

Re: Review Request 69351: SENTRY-2458: Split web service from server service modules

2018-11-30 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69351/#review210982 --- sentry-dist/src/license/THIRD-PARTY.properties Line 34 (original)

Re: Review Request 69415: SENTRY-2463: Revoking ALL or * should revoke any other privilege on the entity

2018-11-30 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69415/ --- (Updated Nov. 30, 2018, 5:21 p.m.) Review request for sentry, Haley Reeve, kaly

Re: Review Request 69087: SENTRY-2249: Enable batch insert of HMS paths in Full Snapshot.

2018-11-30 Thread Na Li via Review Board
> On Nov. 29, 2018, 6:44 p.m., Na Li wrote: > > sentry-service/sentry-service-server/src/main/java/org/apache/sentry/provider/db/service/model/MAuthzPathsMapping.java > > Lines 176 (patched) > > > > > > do you want

Re: Review Request 69351: SENTRY-2458: Split web service from server service modules

2018-11-30 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69351/#review210980 --- Ship it! Ship It! - kalyan kumar kalvagadda On Nov. 29, 2018

Sentry Backup/Restore

2018-11-30 Thread Kalyan Kumar Kalvagadda
Hello all, Currently sentry doesn’t have solution to backup the sentry permission information and restore them in the same OR new cluster. This capability would help in solving disaster recovery and also replication use cases. This means that Sentry should be able to replicate the permission infor

Re: Review Request 68547: SENTRY-2370: Create a cache of PathUpdates to send to NN

2018-11-30 Thread Arjun Mishra via Review Board
> On Sept. 28, 2018, 4:56 p.m., Sergio Pena wrote: > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java > > Lines 136-137 (patched) > > > > > > Why is the cache needed to be inst