[HELP] SENTRY: ALTER_TABLE evnet does not work as expected

2017-12-05 Thread An Qin
Hi, I am working with Sentry + Hive Server. I want to disable the insert permission of user jim who can only select from a table. I implemented my own class that extends org.apache.sentry.binding.hive.v2.metastore.MetastoreAuthzBindingV2. I understand I should response Hive Events: READ_TABLE,

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-05 Thread Zachary Amsden via Review Board
> On Dec. 5, 2017, 8:28 p.m., Na Li wrote: > > sentry-provider/sentry-provider-common/src/main/java/org/apache/sentry/provider/common/ResourceAuthorizationProvider.java > > Line 101 (original), 108 (patched) > > > > > >

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Vadim Spector via Review Board
> On Dec. 5, 2017, 11:25 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Line 420 (original), 430 (patched) > > > > > > Use `

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Vadim Spector via Review Board
> On Dec. 5, 2017, 11:25 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Lines 377 (patched) > > > > > > Will someone up the

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review192945 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Need permissions to update upstream wiki

2017-12-05 Thread Kalyan Kumar Kalvagadda
I would like to make changes to upstream wiki page. https://cwiki.apache.org/confluence/display/SENTRY/How+to+Release but I do not have permissions to do it. It would be great if any one who is authorized to grant me permission can help me with that. -Kalyan

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review192934 --- Ship it! Ship It! - Vadim Spector On Dec. 5, 2017, 9:02 p.m.

Re: Review Request 63958: SENTRY-2034: Add e2e tests for testing HMS notification processing.

2017-12-05 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63958/#review191548 --- sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/te

Re: Review Request 63958: SENTRY-2034: Add e2e tests for testing HMS notification processing.

2017-12-05 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63958/ --- (Updated Dec. 5, 2017, 9:09 p.m.) Review request for sentry, Na Li and Sergio P

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Dec. 5, 2017, 9:02 p.m.) Review request for sentry, kalyan kumar kalva

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-05 Thread Na Li via Review Board
> On Dec. 5, 2017, 8:28 p.m., Na Li wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBindingHookBase.java > > Line 828 (original), 829 (patched) > > > > > > In

Re: Review Request 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-05 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/#review192925 --- sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentr

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review192914 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 63881: SENTRY-2040: When getting Snapshots from HMS we need more logging around cases when a snapshot is not being received

2017-12-05 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Dec. 5, 2017, 5:32 p.m.) Review request for sentry, kalyan kumar kalva

Re: [VOTE] Sentry Release 2.0.0 RC3

2017-12-05 Thread Colm O hEigeartaigh
+1 from me. Checked digests and signatures, built the tag and source distribution, tested the binary distribution. Colm. On Tue, Dec 5, 2017 at 4:30 PM, Sergio Pena wrote: > I verified the following on the RC3: > >- GPG and checksums verified >- Source can be build >- NOTICE, LICENS

Re: [VOTE] Sentry Release 2.0.0 RC3

2017-12-05 Thread Sergio Pena
I verified the following on the RC3: - GPG and checksums verified - Source can be build - NOTICE, LICENSE and CHANGELOG files verified - Install the binary in a local cluster environment with: - hive 2.3.2 - hadoop 2.7.4 - zookeeper 3.4.5 - Sentry HA and Hive autho