----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46957/#review131741 -----------------------------------------------------------
Ship it! Ship It! - Anne Yu On May 4, 2016, 1:58 a.m., Hao Hao wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/46957/ > ----------------------------------------------------------- > > (Updated May 4, 2016, 1:58 a.m.) > > > Review request for sentry and Anne Yu. > > > Repository: sentry > > > Description > ------- > > Sentry-1234: JDO exception for list_sentry_privileges_by_authorizable > > Fix JDO exception for list_sentry_privileges_by_authorizable, and reorganized > the logic for list_sentry_privileges_by_authorizable. > > Change-Id: Ia75e89b7d361d5ff659afe3f1d786fab99089e79 > > > Diffs > ----- > > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/persistent/DelegateSentryStore.java > d51b3baf58de85633755598b55d78d6488fc5041 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericPolicyProcessor.java > 2a287e9b95e36b7d365de603d94208bad59966bc > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClient.java > 60502895a0fcdd781f5bd61b29a676a7c96f81b8 > > sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/generic/service/thrift/SentryGenericServiceClientDefaultImpl.java > dce3dade7f42fe35a849612c5caf2e98d2dac578 > > sentry-provider/sentry-provider-db/src/test/java/org/apache/sentry/provider/db/generic/service/thrift/TestSentryGenericServiceIntegration.java > fcf0e7b9dc45eb12cc1b76f3084efd6340c039a4 > > Diff: https://reviews.apache.org/r/46957/diff/ > > > Testing > ------- > > Added more test cases in TestSentryGenericServiceIntegration. > > > Thanks, > > Hao Hao > >
