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

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

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

2017-02-27 Thread Alexander Kolbasov
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/#review167014 --- Fix it, then Ship it! Ship It!

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

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

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

2017-02-22 Thread Lei Xu
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/56952/ --- Review request for sentry. Bugs: SENTRY-1637