Re: Review Request 64955: SENTRY-2109: Fix the logic of identifying HMS out of Sync

2018-01-08 Thread kalyan kumar kalvagadda via Review Board
> On Jan. 4, 2018, 10:40 p.m., Vadim Spector wrote: > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HMSFollower.java > > Line 205 (original), 206 (patched) > > > > > > I thought

Review Request 65028: SENTRY-1819 HMSFollower and friends do not belong in sentry.service.thrift

2018-01-08 Thread Xinran Tinney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65028/ --- Review request for sentry, Alexander Kolbasov, Arjun Mishra, Colm O hEigeartaigh

Re: Review Request 64949: SENTRY-641 Add binding for lily hbase indexer

2018-01-08 Thread Mano Kovacs via Review Board
> On Jan. 4, 2018, 10:19 p.m., Sergio Pena wrote: > > sentry-binding/sentry-binding-hbase-indexer/src/main/java/org/apache/sentry/binding/hbaseindexer/authz/HBaseIndexerAuthzBinding.java > > Lines 199 (patched) > > > > >

Re: Review Request 64955: SENTRY-2109: Fix the logic of identifying HMS out of Sync

2018-01-08 Thread Arjun Mishra via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64955/#review194953 --- sentry-provider/sentry-provider-db/src/main/java/org/apache/sentr

Re: Review Request 64960: SENTRY-2113: MSentryHmsNotification should also hold the notification hash.

2018-01-08 Thread kalyan kumar kalvagadda via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64960/ --- (Updated Jan. 8, 2018, 3:26 p.m.) Review request for sentry, Alexander Kolbasov