-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47630/#review134084
-----------------------------------------------------------




sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart1.java
 (line 488)
<https://reviews.apache.org/r/47630/#comment198729>

    Better to create a new @Test


please also check authzV2 can pass the Test

- Dapeng Sun


On 五月 20, 2016, 10:47 a.m., Ke Jia wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/47630/
> -----------------------------------------------------------
> 
> (Updated 五月 20, 2016, 10:47 a.m.)
> 
> 
> Review request for sentry, Colin Ma and Dapeng Sun.
> 
> 
> Repository: sentry
> 
> 
> Description
> -------
> 
> Enable ALTERVIEW_RENAME in hive binding
> 
> 
> Diffs
> -----
> 
>   
> sentry-binding/sentry-binding-hive/src/main/java/org/apache/sentry/binding/hive/authz/HiveAuthzPrivilegesMap.java
>  bc158ff 
>   
> sentry-tests/sentry-tests-hive/src/test/java/org/apache/sentry/tests/e2e/hive/TestOperationsPart1.java
>  0522ba0 
> 
> Diff: https://reviews.apache.org/r/47630/diff/
> 
> 
> Testing
> -------
> 
> all test passed
> 
> 
> Thanks,
> 
> Ke Jia
> 
>

Reply via email to