-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29393/
-----------------------------------------------------------
(Updated 一月 28, 2015, 3:25 p.m.)
Review request for sentry, Xiaomeng Huang, Arun Suresh, Colin Ma, shen guoquan,
Lenni Kuff, Prasad Mujumdar, and Sravya Tirukkovalur.
Changes
-------
Accroding Lenni's feedback, remove the change "90s" in
SentryServiceIntegrationBase, thank Lenni for the review.
Bugs: SENTRY-599
https://issues.apache.org/jira/browse/SENTRY-599
Repository: sentry
Description
-------
The patch is fixing a issue that, when sentry service is restarting using same
configuration and context. ** server.isRunning()** will report incorrect status.
The root cause is **finally** may run after **future =
serviceExecutor.submit(this);**, the status will be **NOT_STARTED** even the
thrift server is started
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
02788aa
sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/service/thrift/TestSentryServiceIntegration.java
d4dfa23
Diff: https://reviews.apache.org/r/29393/diff/
Testing
-------
Unit Tests passed in apache jenkins
Thanks,
Dapeng Sun
