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

[Haley Reeve via Review Board]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/69596/
-----------------------------------------------------------

(Updated Jan. 4, 2019, 4:33 p.m.)


Review request for sentry, Arjun Mishra, kalyan kumar kalvagadda, Na Li, and 
Sergio Pena.


Bugs: SENTRY-1797
    https://issues.apache.org/jira/browse/SENTRY-1797


Repository: sentry


Description
-------

The SentryKerberosContext class tries to simulate periodic executor using while 
loop with a sleep in the run() method. Instead it should use periodic executor.


Diffs (updated)
-----

  
sentry-service/sentry-service-server/src/main/java/org/apache/sentry/service/thrift/SentryKerberosContext.java
 efb8ae6 


Diff: https://reviews.apache.org/r/69596/diff/2/

Changes: https://reviews.apache.org/r/69596/diff/1-2/


Testing
-------

Ran unit tests.


Thanks,

Haley Reeve