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 2014/06/08 22:00:22 UTC
Review Request 22357: SENTRY-251: PolicyProviderForTest.addPrivilege breaks
in some cases
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22357/
-----------------------------------------------------------
Review request for sentry, Jarek Cecho and Prasad Mujumdar.
Bugs: sentry-251 and sentry-252
https://issues.apache.org/jira/browse/sentry-251
https://issues.apache.org/jira/browse/sentry-252
Repository: sentry
Description
-------
Minor fix to allow multiple privileges to be specified on the same line.
Diffs
-----
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/PolicyProviderForTest.java bc2920d65cadfc7f7cb00cb8c0720a897f244968
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java 5eef7924aeeb5ad66566df3f9a67d552025370a2
Diff: https://reviews.apache.org/r/22357/diff/
Testing
-------
Tests TestDbSandboxOps.testInsertOverwriteAndLoadData and TestDbUriPermissions.testLoadPrivileges pass.
Thanks,
Sravya Tirukkovalur
Re: Review Request 22357: SENTRY-251: PolicyProviderForTest.addPrivilege
breaks in some cases
Posted by Jarek Cecho <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22357/#review45042
-----------------------------------------------------------
Ship it!
Ship It!
- Jarek Cecho
On June 8, 2014, 8 p.m., Sravya Tirukkovalur wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/22357/
> -----------------------------------------------------------
>
> (Updated June 8, 2014, 8 p.m.)
>
>
> Review request for sentry, Jarek Cecho and Prasad Mujumdar.
>
>
> Bugs: sentry-251 and sentry-252
> https://issues.apache.org/jira/browse/sentry-251
> https://issues.apache.org/jira/browse/sentry-252
>
>
> Repository: sentry
>
>
> Description
> -------
>
> Minor fix to allow multiple privileges to be specified on the same line.
>
>
> Diffs
> -----
>
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/PolicyProviderForTest.java bc2920d65cadfc7f7cb00cb8c0720a897f244968
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/dbprovider/TestDbSandboxOps.java 5eef7924aeeb5ad66566df3f9a67d552025370a2
>
> Diff: https://reviews.apache.org/r/22357/diff/
>
>
> Testing
> -------
>
> Tests TestDbSandboxOps.testInsertOverwriteAndLoadData and TestDbUriPermissions.testLoadPrivileges pass.
>
>
> Thanks,
>
> Sravya Tirukkovalur
>
>