You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Mahendra Korepu (Code Review)" <ge...@cloudera.org> on 2019/03/05 18:12:38 UTC

[Impala-ASF-CR] IMPALA-8269: Clean up authorization test package structure

Mahendra Korepu has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/12654 )

Change subject: IMPALA-8269: Clean up authorization test package structure
......................................................................

IMPALA-8269: Clean up authorization test package structure

1.Moved AuthorizatioinTest.java and AuthorizationStmtTest.java to
authorization test package.
2.Rename CustomClusterGroupMapper and
CustomClusterResourceAuthorizationProvider
to TestSentryGroupMapper and TestSentryResourceAuthorizationProvider
since
those two class aren't specific to custom cluster anymore.
3.Moved above files to testutil.
4.Updated python tests with renamed java class references.

Testing:
 - Ran all Authorizatioin tests

Change-Id: I575b64d3d95445aaa9735a80d0aaca4b6714e17b
---
R fe/src/test/java/org/apache/impala/authorization/AuthorizationStmtTest.java
R fe/src/test/java/org/apache/impala/authorization/AuthorizationTest.java
R fe/src/test/java/org/apache/impala/testutil/TestSentryGroupMapper.java
R fe/src/test/java/org/apache/impala/testutil/TestSentryResourceAuthorizationProvider.java
M fe/src/test/resources/sentry-site_no_oo.xml.template
M fe/src/test/resources/sentry-site_oo.xml.template
M fe/src/test/resources/sentry-site_oo_nogrant.xml.template
M tests/authorization/test_grant_revoke.py
M tests/authorization/test_owner_privileges.py
M tests/authorization/test_show_grant.py
10 files changed, 33 insertions(+), 30 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/54/12654/2
-- 
To view, visit http://gerrit.cloudera.org:8080/12654
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I575b64d3d95445aaa9735a80d0aaca4b6714e17b
Gerrit-Change-Number: 12654
Gerrit-PatchSet: 2
Gerrit-Owner: Mahendra Korepu <ko...@gmail.com>
Gerrit-Reviewer: Austin Nobis <an...@cloudera.com>
Gerrit-Reviewer: Fredy Wijaya <fw...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <im...@cloudera.com>