You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sentry.apache.org by Sravya Tirukkovalur <sr...@cloudera.com> on 2015/06/01 20:20:33 UTC
Re: Review Request 31074: Sentry-657: Update SentryConfigTool for
import/export feature
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31074/#review85686
-----------------------------------------------------------
Ship it!
Looks good to me, thanks Colin!
- Sravya Tirukkovalur
On May 20, 2015, 1:35 a.m., Colin Ma wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31074/
> -----------------------------------------------------------
>
> (Updated May 20, 2015, 1:35 a.m.)
>
>
> Review request for sentry.
>
>
> Repository: sentry
>
>
> Description
> -------
>
> Update SentryConfigTool for import/export feature
>
>
> Diffs
> -----
>
> sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/SentryConfigTool.java ecbd664
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImport.java 7ebc0e4
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestPolicyImportExport.java PRE-CREATION
> sentry-tests/sentry-tests-hive/src/test/resources/testPolicyImport.ini PRE-CREATION
> sentry-tests/sentry-tests-hive/src/test/resources/testPolicyImportAdmin.ini PRE-CREATION
> sentry-tests/sentry-tests-hive/src/test/resources/testPolicyImportError.ini PRE-CREATION
>
> Diff: https://reviews.apache.org/r/31074/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Colin Ma
>
>