[
https://issues.apache.org/jira/browse/IMPALA-8269?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16832083#comment-16832083
]
ASF subversion and git services commented on IMPALA-8269:
---------------------------------------------------------
Commit d29d68c90d557353f7e312ba99747a6adca23820 in impala's branch
refs/heads/master from Fredy Wijaya
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=d29d68c ]
IMPALA-8269: Refactor authorization test package structure
This patch refactors authorization test package structure by moving it
to the authorization test package. This patch also renames
CustomClusterResourceAuthorizationProvider to
TestSentryResourceAuthorizationProvider since it is a class that is
specific to Sentry and is only used for testing.
Testing:
- Ran all FE tests
- Ran all E2E authorization tests
Change-Id: I525ff71f63d7c306d82b4c111f98ff327e4a07b3
Reviewed-on: http://gerrit.cloudera.org:8080/13208
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Clean up authorization test package structure
> ---------------------------------------------
>
> Key: IMPALA-8269
> URL: https://issues.apache.org/jira/browse/IMPALA-8269
> Project: IMPALA
> Issue Type: Improvement
> Components: Infrastructure
> Reporter: Fredy Wijaya
> Assignee: Fredy Wijaya
> Priority: Minor
> Labels: ramp-up
> Fix For: Impala 3.3.0
>
>
> The task is to do some clean up on the authorization test package structure.
> 1. Move 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. Move those two files into `testutil` instead since they're not actually
> test classes.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
