Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-12 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55246/ --- (Updated Jan. 13, 2017, 3:06 a.m.) Review request for sentry, Alexander

Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-12 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55246/#review161477 --- I would consider changing your interface a bit - instead of

Re: Review Request 55094: SENTRY-1532: Sentry Web UI isn't working

2017-01-12 Thread Alexander Kolbasov
> On Jan. 12, 2017, 11:46 p.m., kalyan kumar kalvagadda wrote: > > sentry-service/sentry-service-server/pom.xml, line 117 > > > > > > I understand that it was working before this changes.It should be > > working

Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-12 Thread Lei Xu
> On Jan. 11, 2017, 1:21 p.m., Lei Xu wrote: > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java, > > line 446 > > > > > > Lets not move this. > > > > Btw, will

Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-12 Thread Hao Hao
> On Jan. 12, 2017, 1:35 a.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/SentryStore.java, > > line 106 > > > > > > final? Also,

Re: Review Request 55246: SENTRY-1536: Refactor SentryStore transaction management to allow for extra transactions for a single permission update

2017-01-12 Thread Hao Hao
> On Jan. 11, 2017, 1:21 p.m., Lei Xu wrote: > > sentry-hdfs/sentry-hdfs-service/src/main/java/org/apache/sentry/hdfs/SentryPlugin.java, > > line 327 > > > > > > Could you explain why here we create a local map, as

Re: Review Request 55094: SENTRY-1532: Sentry Web UI isn't working

2017-01-12 Thread kalyan kumar kalvagadda
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55094/#review161459 --- sentry-service/sentry-service-server/pom.xml (line 117)

FOSDEM 2017 Open Source Conference - Brussels

2017-01-12 Thread Sharan F
Hello Everyone This email is to tell you about ASF participation at FOSDEM. The event will be held in Brussels on 4^th & 5^th February 2017 and we are hoping that many people from our ASF projects will be there. https://fosdem.org/2017/ Attending FOSDEM is completely free and the ASF will