Re: Review Request 64392: SENTRY-2092: Drop Role log message shows Creating role

2017-12-06 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64392/#review193061 --- Ship it! Ship It! - kalyan kumar kalvagadda On Dec. 6, 2017,

Re: Review Request 64392: SENTRY-2092: Drop Role log message shows Creating role

2017-12-06 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64392/#review193060 --- Ship it! Ship It! - Alexander Kolbasov On Dec. 6, 2017, 11:4

Re: Review Request 64392: SENTRY-2092: Drop Role log message shows Creating role

2017-12-06 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64392/ --- (Updated Dec. 6, 2017, 11:42 p.m.) Review request for sentry, kalyan kumar kalv

Re: [VOTE] Sentry Release 2.0.0 RC3

2017-12-06 Thread Alexander Kolbasov
+1 On Tue, Dec 5, 2017 at 9:01 AM, Colm O hEigeartaigh wrote: > +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: > >

Re: Review Request 64392: SENTRY-2092: Drop Role log message shows Creating role

2017-12-06 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64392/#review193051 --- sentry-binding/sentry-binding-hive-v2/src/main/java/org/apache/se

Review Request 64392: SENTRY-2092: Drop Role log message shows Creating role

2017-12-06 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64392/ --- Review request for sentry, kalyan kumar kalvagadda and Sergio Pena. Repository:

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

2017-12-06 Thread Na Li via Review Board
> On Nov. 21, 2017, 8:37 p.m., Na Li wrote: > > sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestHmsNotificationProcessingWithOutSyncOnCreate.java > > Lines 81 (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-06 Thread Vadim Spector via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/#review193045 --- 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-06 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Dec. 6, 2017, 9:03 p.m.) Review request for sentry, kalyan kumar kalva

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-06 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Dec. 6, 2017, 8:25 p.m.) Review request for sentry, kalyan kumar kalva

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

2017-12-06 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64317/#review193011 --- Ship it! Ship It! - Na Li On Dec. 5, 2017, 12:55 a.m., Zacha

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

2017-12-06 Thread Na Li 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 64317: SENTRY-2085: Keep sentry exceptions contained within Sentry

2017-12-06 Thread Na Li 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: Need permissions to update upstream wiki

2017-12-06 Thread Colm O hEigeartaigh
What's your cwiki username though? Colm. On Wed, Dec 6, 2017 at 4:56 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Colm, > > My apache user name is just kalyan. > > -Kalyan > > On Wed, Dec 6, 2017 at 3:55 AM, Colm O hEigeartaigh > wrote: > > > I've granted permissions to the user

Re: Need permissions to update upstream wiki

2017-12-06 Thread Kalyan Kumar Kalvagadda
Colm, My apache user name is just kalyan. -Kalyan On Wed, Dec 6, 2017 at 3:55 AM, Colm O hEigeartaigh wrote: > I've granted permissions to the user "kkalyan" (let me know if this is the > wrong username!). You might have to log off and log in again for it to > work. > > Colm. > > On Tue, Dec 5

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-06 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63881/ --- (Updated Dec. 6, 2017, 4:01 p.m.) Review request for sentry, kalyan kumar kalva

Re: Need permissions to update upstream wiki

2017-12-06 Thread Colm O hEigeartaigh
I've granted permissions to the user "kkalyan" (let me know if this is the wrong username!). You might have to log off and log in again for it to work. Colm. On Tue, Dec 5, 2017 at 9:20 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > I would like to make changes to upstream wiki pag