Review Request 57065: SENTRY-1600 Define Thrift API for HMS to Sentry notification barrier

2017-02-24 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57065/ --- Review request for sentry, Hao Hao, kalyan kumar kalvagadda, Sergio Pena, and Va

Sentry-jdk-1.7-v2 - Build # 66 - Fixed

2017-02-24 Thread Apache Jenkins Server
The Apache Jenkins build system has built Sentry-jdk-1.7-v2 (build #66) Status: Fixed Check console output at https://builds.apache.org/job/Sentry-jdk-1.7-v2/66/ to view the results.

Re: Review Request 56481: SENTRY-1625: PrivilegeOperatePersistence can use QueryParamBuilder

2017-02-24 Thread Vadim Spector
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56481/#review166783 --- Ship it! - Vadim Spector On Feb. 23, 2017, 8:30 p.m., Alexand

Re: Review Request 56481: SENTRY-1625: PrivilegeOperatePersistence can use QueryParamBuilder

2017-02-24 Thread Hao Hao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56481/#review166775 --- Ship it! Ship It! - Hao Hao On Feb. 23, 2017, 8:30 p.m., Ale

Re: Review Request 56952: SENTRY-1637. Periodically purge Delta change tables.

2017-02-24 Thread Lei Xu
> On Feb. 23, 2017, 4:43 p.m., Alexander Kolbasov wrote: > > I would suggest a simple approach: > > > > 1) Remove all knowledge about the priv cleaner from SentryStore itself. > > 2) Priv cleaner needs SentryStore for DB operations, so we should create > > one in SentryService - it will be shar

Re: Review Request 56952: SENTRY-1637. Periodically purge Delta change tables.

2017-02-24 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/ --- (Updated Feb. 24, 2017, 2:47 p.m.) Review request for sentry. Changes ---