[
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.
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
> 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: [email protected]
For additional commands, e-mail: [email protected]
