----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/46957/ -----------------------------------------------------------
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
