----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30839/#review72986 -----------------------------------------------------------
Ship it! Ship It! - Lenni Kuff On Feb. 12, 2015, 8:40 a.m., Prasad Mujumdar wrote: > > ----------------------------------------------------------- > 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. > > > 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 > ----- > > > 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 > >
