[
https://issues.apache.org/jira/browse/IMPALA-8248?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
radford nguyen updated IMPALA-8248:
-----------------------------------
Description:
We have authorization tests that are specific to Sentry and authorization tests
that can be applicable to any authorization provider. We need to re-organize
the authorization tests to easily differentiate between Sentry-specific tests
vs generic authorization tests.
h3. Approach
# Move `AuthorizationTest.java` and `AuthorizationStmtTest.java` to
`org.apache.impala.authorization`
# Rename `CustomClusterGroupMapper` and
`CustomClusterResourceAuthorizationProvider` to `TestSentryGroupMapper` and
`TestSentryAuthorizationProvider` since those two class aren't specific to
custom cluster anymore.
# Move those two files into `org.apache.impala.testutil` instead since they're
not actually test classes.
Note: all classes to remain in `test` sourceset
was:We have authorization tests that are specific to Sentry and authorization
tests that can be applicable to any authorization provider. We need to
re-organize the authorization tests to easily differentiate between
Sentry-specific tests vs generic authorization tests.
> Re-organize authorization tests
> -------------------------------
>
> Key: IMPALA-8248
> URL: https://issues.apache.org/jira/browse/IMPALA-8248
> Project: IMPALA
> Issue Type: Sub-task
> Components: Infrastructure
> Reporter: Fredy Wijaya
> Assignee: radford nguyen
> Priority: Major
>
> We have authorization tests that are specific to Sentry and authorization
> tests that can be applicable to any authorization provider. We need to
> re-organize the authorization tests to easily differentiate between
> Sentry-specific tests vs generic authorization tests.
>
> h3. Approach
> # Move `AuthorizationTest.java` and `AuthorizationStmtTest.java` to
> `org.apache.impala.authorization`
> # Rename `CustomClusterGroupMapper` and
> `CustomClusterResourceAuthorizationProvider` to `TestSentryGroupMapper` and
> `TestSentryAuthorizationProvider` since those two class aren't specific to
> custom cluster anymore.
> # Move those two files into `org.apache.impala.testutil` instead since
> they're not actually test classes.
> Note: all classes to remain in `test` sourceset
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org
