You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Hao Hao <ha...@cloudera.com> on 2016/05/04 03:58:24 UTC
Review Request 46957: Sentry-1234: JDO exception for
list_sentry_privileges_by_authorizable
-----------------------------------------------------------
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
Re: Review Request 46957: Sentry-1234: JDO exception for
list_sentry_privileges_by_authorizable
Posted by Anne Yu <an...@cloudera.com>.
-----------------------------------------------------------
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
>
>