Review Request 59567: SENTRY-1783 alterSentryRoleGrantPrivilegeCore does more persistence work than required

2017-05-24 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59567/ --- Review request for sentry, Brian Towles, Hao Hao, kalyan kumar kalvagadda, Na Li

Review Request 59566: SENTRY-1784: DBUpdateForwarder returns empty update list to HDFS instead of full update

2017-05-24 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59566/ --- Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, Sergio P

Re: Review Request 59414: SENTRY-1771: HDFS client requests full permission update multiple times.

2017-05-24 Thread Na Li
> On May 19, 2017, 8:22 p.m., Sergio Pena wrote: > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java > > Lines 217 (patched) > > > > > > If the thread is sche

Re: Review Request 59414: SENTRY-1771: HDFS client requests full permission update multiple times.

2017-05-24 Thread Na Li
> On May 19, 2017, 8:22 p.m., Sergio Pena wrote: > > sentry-hdfs/sentry-hdfs-namenode-plugin/src/main/java/org/apache/sentry/hdfs/SentryAuthorizationInfo.java > > Line 183 (original), 185 (patched) > > > > > > I read

Re: Review Request 59510: SENTRY-1774 HMSFollower should always depend on persisted information to decide is full snapshot is needed

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

Re: Review Request 59167: SENTRY-1580: Provide pooled client connection model with HA

2017-05-24 Thread Alexander Kolbasov
> On May 24, 2017, 12:10 p.m., kalyan kumar kalvagadda wrote: > > Seems like you have an emty comment. - Alexander --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/#review175920

Re: Review Request 59508: SENTRY-1769 Refactor HMSFollower Class

2017-05-24 Thread Sergio Pena
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59508/#review175955 --- A few comments Kalyan. I need to do more review, but I wanted to l

Re: [DISCUSS] Considering Sentry release

2017-05-24 Thread Sergio Pena
Sounds interesting Sasha. +1 (non-binding) I took a look at the # of commits between 1.7.0 and master and there are currently 154 new commits on master. This is good enough to do a release for Sentry. About Sentry HA, it makes sense to move this work to a major Sentry release, such as 2.0 due to

[DISCUSS] Considering Sentry release

2017-05-24 Thread Alexander Kolbasov
I would like to start a preliminary discussion about the new Sentry release. To make this interesting, I suggest to consider thinking about two releases. 1) There are a lot of accumulated fixes since 1.7. IMO it would be useful to release those as 1.8 release. 2) Sentry HA includes a large numb

Re: Information about 1.7 release is missing from the wiki

2017-05-24 Thread Sergio Pena
I can update it. How do I get permissions to the svn repo? On Tue, May 23, 2017 at 6:01 PM, Alexander Kolbasov wrote: > I noticed that the wiki at https://sentry.apache.org/general/history.html > is missing information > about 1.7 release. Would b

Re: Review Request 59508: SENTRY-1769 Refactor HMSFollower Class

2017-05-24 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59508/#review175941 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 59510: SENTRY-1774 HMSFollower should always depend on persisted information to decide is full snapshot is needed

2017-05-24 Thread Na Li
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59510/#review175939 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 59167: SENTRY-1580: Provide pooled client connection model with HA

2017-05-24 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/#review175920 --- - kalyan kumar kalvagadda On May 24, 2017, 7:49 a.m., Alexander

Re: Who maintains sentry.apache.org?

2017-05-24 Thread Colm O hEigeartaigh
Hi Alex, It's generated from content applied to: https://svn.apache.org/repos/asf/sentry/site/trunk/ Do you have write access to that repo? Colm. On Wed, May 24, 2017 at 12:02 AM, Alexander Kolbasov wrote: > Who has write access to https://sentry.apache.org/index.html? < > https://sentry.apac

Re: Review Request 59167: SENTRY-1580: Provide pooled client connection model with HA

2017-05-24 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/ --- (Updated May 24, 2017, 7:49 a.m.) Review request for sentry, Brian Towles, Hao