[ 
https://issues.apache.org/jira/browse/IMPALA-6802?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16791323#comment-16791323
 ] 

ASF subversion and git services commented on IMPALA-6802:
---------------------------------------------------------

Commit 139f3b7351bf6a45689b670fa3d388d56cf4cc72 in impala's branch 
refs/heads/2.x from Fredy Wijaya
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=139f3b7 ]

IMPALA-6802 (part 6): Clean up authorization tests

This is the last part of the authorization test clean up.

This patch rewrites the following tests:
- alter database
- explain
- comment on
- function
- alter table/view

This patch also adds the following authorization tests:
- update
- upsert
- delete

The tests in AuthorizationTest.java that have been rewritten into
AuthorizationStmtTest.java are removed.

Cherry-picks: not for 2.x

Change-Id: Id594ce09a821aef4a1debfdd61569a11defd1c55
Reviewed-on: http://gerrit.cloudera.org:8080/10841
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-on: http://gerrit.cloudera.org:8080/12722
Reviewed-by: Fredy Wijaya <[email protected]>


> Clean up tests in AuthorizationTest
> -----------------------------------
>
>                 Key: IMPALA-6802
>                 URL: https://issues.apache.org/jira/browse/IMPALA-6802
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Infrastructure
>            Reporter: Fredy Wijaya
>            Assignee: Fredy Wijaya
>            Priority: Critical
>              Labels: security
>             Fix For: Impala 2.13.0, Impala 3.1.0
>
>
> The tests in AuthorizationTest can be refactored to be more condensed. We 
> also need to increase the code coverage. It is also a good idea to refactor 
> these tests to provide flexibility in supporting tests for user-level 
> privileges in the future: https://issues.apache.org/jira/browse/IMPALA-6794



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to