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

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

Review Request 59174: SENTRY-1761 Add test for concurrent HMS followers processing notifications

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59174/ --- Review request for sentry, Alexander Kolbasov, Brian Towles, Hao Hao, Na Li, Ser

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/ --- (Updated May 11, 2017, 12:55 p.m.) Review request for sentry, Alexander Kolbaso

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

> On May 10, 2017, 10:29 p.m., Alexander Kolbasov wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/DeltaTransactionBlock.java > > Lines 91 (patched) > > > > > >

Re: Review Request 58808: SENTRY-1726 sql changes to store last notification-id processed

> On May 10, 2017, 6:19 p.m., Alexander Kolbasov wrote: > > You have only one upgrade script updated - do you need to update other > > upgrade scripts or not? I have used existing script so I need not change the upgrade sript. - kalyan kumar -

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/ --- (Updated May 11, 2017, 1:16 p.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59077: SENTRY-1739 Sentry Kafka tests do not stop periodic update after the test end

> On May 9, 2017, 3:22 p.m., Na Li wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/UpdatableCache.java > > Lines 39 (patched) > > > > > > Could several threads create

Review Request 59179: SENTRY-1756: Passive nodes should still follow latest notification ID

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59179/ --- Review request for sentry. Bugs: SENTRY-1756 https://issues.apache.org/jira

Re: Review Request 59084: SENTRY-1516 - Add gpg configuration to the root pom to enable deployment to Maven Central

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59084/#review174669 --- Colm, the changes look good. Just a quetion, how do I test this ch

Review Request 59204: SENTRY-1717: Sentry should emit log messages when it is ready to serve requests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59204/ --- Review request for sentry, Alexander Kolbasov, kalyan kumar kalvagadda, and Serg

[DISCUSS] Having an interactive chat account for Apache Sentry

What do people think about having Apache Sentry chat room where we can hang out and informally chat and discuss technical and non-technical issues? Is there any Apache infrastructure for that (e.g. HipChat or Slack account)? If not we can use some private account for that purpose (I think that’s

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

Sasha, That an nice idea. Do you know what Apache infrastructure is kudu project using? -Kalyan -Kalyan On Thu, May 11, 2017 at 6:12 PM, Alex Kolbasov wrote: > What do people think about having Apache Sentry chat room where we can > hang out and informally chat and discuss technical and non-t

Review Request 59212: SENTRY-1763 Fix the config string for server load balancing

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59212/ --- Review request for sentry and Alexander Kolbasov. Bugs: SENTRY-1763 https:/

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

+1. IIRC, they are using slack. On Thu, May 11, 2017 at 4:21 PM, Kalyan Kumar Kalvagadda < kkal...@cloudera.com> wrote: > Sasha, > > That an nice idea. Do you know what Apache infrastructure is kudu project > using? > > -Kalyan > > -Kalyan > > On Thu, May 11, 2017 at 6:12 PM, Alex Kolbasov wrote

Re: [DISCUSS] Having an interactive chat account for Apache Sentry

As far as I know Kudu isn’t using Apache infrastructure, they are using private account and Slack integration: https://github.com/rauchg/slackin I think I can have a free private account on Slack with some reasonable limitations if that’s ok for others. I am

Re: Review Request 58975: SENTRY-1669 HMSFollower should read current processed notification ID from database every time it runs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58975/#review174744 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 58808: SENTRY-1726 sql changes to store last notification-id processed

> On April 27, 2017, 8:25 p.m., Hao Hao wrote: > > sentry-provider/sentry-provider-db/src/main/resources/008-SENTRY-1569.derby.sql > > Lines 30 (patched) > > > > > > Can you explain more about why using 'signleton'and

Re: Review Request 58808: SENTRY-1726 sql changes to store last notification-id processed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58808/#review174749 --- Ship it! Ship It! - Hao Hao On May 9, 2017, 11:15 p.m., kaly

Re: Review Request 58808: SENTRY-1726 sql changes to store last notification-id processed

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58808/#review174750 --- Ship it! Ship It! - Vamsee Yarlagadda On May 9, 2017, 11:15

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

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59167/#review174745 --- sentry-binding/sentry-binding-hive-common/src/main/java/org/apach

Re: Review Request 59120: SENTRY-1757: Avoid using local hive meta store using wrong configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59120/ --- (Updated May 12, 2017, 3:40 a.m.) Review request for sentry, Alexander Kolbasov

Re: Review Request 59120: SENTRY-1757: Avoid using local hive meta store using wrong configuration

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59120/#review174758 --- sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/te

Re: Review Request 59204: SENTRY-1717: Sentry should emit log messages when it is ready to serve requests

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59204/#review174759 --- Ship it! Ship It! - Alexander Kolbasov On May 11, 2017, 8:54