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/05/20 23:10:03 UTC
Review Request 21733: SENTRY-220: Trivial fix to SentrySchemaTool to set
default driver
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21733/
-----------------------------------------------------------
Review request for sentry and Prasad Mujumdar.
Repository: sentry
Description
-------
Trivial fix to SentrySchemaTool to set default driver
Diffs
-----
sentry-binding/sentry-binding-hive/src/main/java/org/apache/hadoop/hive/ql/exec/SentryGrantRevokeTask.java b6f39ad7c4c15b377e7ea55e28ec7bbe19653d18
sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzBinding.java 7a561ef0f4572490b35fc3408bd18fb17ce5f21c
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 1aa767eed659035c37a8c4b093091e4324041de8
sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestDatabaseProvider.java 176aceeeed33302e5f0dc7144b673eb792fb3e3d
Diff: https://reviews.apache.org/r/21733/diff/
Testing
-------
Thanks,
Sravya Tirukkovalur
Re: Review Request 21733: SENTRY-220: Trivial fix to SentrySchemaTool to set
default driver
Posted by Prasad Mujumdar <pr...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21733/#review43601
-----------------------------------------------------------
Ship it!
Ship It!
- Prasad Mujumdar
On May 20, 2014, 9:18 p.m., Sravya Tirukkovalur wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/21733/
> -----------------------------------------------------------
>
> (Updated May 20, 2014, 9:18 p.m.)
>
>
> Review request for sentry and Prasad Mujumdar.
>
>
> Repository: sentry
>
>
> Description
> -------
>
> Trivial fix to SentrySchemaTool to set default driver
>
>
> Diffs
> -----
>
> sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 1aa767eed659035c37a8c4b093091e4324041de8
>
> Diff: https://reviews.apache.org/r/21733/diff/
>
>
> Testing
> -------
>
>
> Thanks,
>
> Sravya Tirukkovalur
>
>
Re: Review Request 21733: SENTRY-220: Trivial fix to SentrySchemaTool to set
default driver
Posted by Sravya Tirukkovalur <sr...@cloudera.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21733/
-----------------------------------------------------------
(Updated May 20, 2014, 9:18 p.m.)
Review request for sentry and Prasad Mujumdar.
Changes
-------
Attached a wrong patch previously.
Repository: sentry
Description
-------
Trivial fix to SentrySchemaTool to set default driver
Diffs (updated)
-----
sentry-provider/sentry-provider-db/src/main/java/org/apache/sentry/provider/db/tools/SentrySchemaTool.java 1aa767eed659035c37a8c4b093091e4324041de8
Diff: https://reviews.apache.org/r/21733/diff/
Testing
-------
Thanks,
Sravya Tirukkovalur