-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30839/
-----------------------------------------------------------
(Updated Feb. 12, 2015, 8:40 a.m.)
Review request for sentry and Dapeng Sun.
Changes
-------
Addressed review feedback
Bugs: SENTRY-648
https://issues.apache.org/jira/browse/SENTRY-648
Repository: sentry
Description
-------
- Update test framework to allow creating multiple services
- Sentry service abstraction to enable start/stop of individual servers from
test
- Added test cases
- Fixed issues ZK session leak found with the new tests
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/service/persistent/ServiceManager.java
2274f00
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/HAClientInvocationHandler.java
c6e265f
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/SentryService.java
02788aa
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/service/thrift/ServiceConstants.java
ddc5930
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/ha/TestHaEnd2End.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/AbstractTestWithStaticConfiguration.java
d08b4ee
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/InternalSentrySrv.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrv.java
PRE-CREATION
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/minisentry/SentrySrvFactory.java
PRE-CREATION
Diff: https://reviews.apache.org/r/30839/diff/
Testing
-------
Added New tests -
- Normal Sentry operations with HA enabled
- Testing service failover
Thanks,
Prasad Mujumdar
