Re: Review Request 69586: SENTRY-2481: Filter HMS server-side objects based on HMS user authorization

2018-12-19 Thread Na Li via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69586/#review211441 --- Ship it! Ship It! - Na Li On Dec. 19, 2018, 3:24 p.m., Sergi

Review Request 69596: SENTRY-1797: SentryKerberosContext should use periodic executor instead of managing periodic execution via run() method.

2018-12-19 Thread Haley Reeve via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69596/ --- Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, Na Li, and Ser

Re: Review Request 69586: SENTRY-2481: Filter HMS server-side objects based on HMS user authorization

2018-12-19 Thread Sergio Pena via Review Board
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69586/ --- (Updated Dec. 19, 2018, 3:24 p.m.) Review request for sentry, Arjun Mishra, kal

Re: Review Request 69586: SENTRY-2481: Filter HMS server-side objects based on HMS user authorization

2018-12-19 Thread Sergio Pena via Review Board
> On Dec. 18, 2018, 7:52 p.m., Na Li wrote: > > sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/DefaultSentryValidator.java > > Lines 67 (patched) > > > > > > If the object is not