[
https://issues.apache.org/jira/browse/IMPALA-9160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17084246#comment-17084246
]
ASF subversion and git services commented on IMPALA-9160:
---------------------------------------------------------
Commit 026ae182ef4b5111aba625a26d50b8b53563088d in impala's branch
refs/heads/master from Fang-Yu Rao
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=026ae18 ]
IMPALA-9160 (Part 2): Instantiate rangerConfig_ using RangerPluginConfig
Now that we are using a CDP_BUILD_NUMBER newer than 1626038, we are able
to instantiate rangerConfig_ in RangerAuthorizationConfig.java using the
class of RangerPluginConfig provided in the newer version of Ranger due
to RANGER-2646. This patch adds back rangerConfig_, which was removed
from RangerAuthorizationConfig.java in part 1 of this patch.
Testing:
- Verified that this patch passes the exhaustive tests in the DEBUG
build.
Change-Id: I710a6362a2494bf5ab3784259a44c3470823512f
Reviewed-on: http://gerrit.cloudera.org:8080/15721
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Remove references to RangerAuthorizationConfig due to changes in Ranger
> -----------------------------------------------------------------------
>
> Key: IMPALA-9160
> URL: https://issues.apache.org/jira/browse/IMPALA-9160
> Project: IMPALA
> Issue Type: Improvement
> Reporter: Fang-Yu Rao
> Assignee: Fang-Yu Rao
> Priority: Major
> Fix For: Impala 4.0
>
>
> Due to recent changes in https://issues.apache.org/jira/browse/RANGER-2646,
> the way Impala instantiates a Ranger authorization configuration in
> {{RangerAuthorizationConfig.java}} should change accordingly.
> Two steps are thus required.
> # Remove the references to the class of {{RangerConfiguration}} in
> {{RangerAuthorizationConfig.java}}.
> # Instantiate {{rangerConfig_}} using the class of {{RangerPluginConfig}}
> provided in the newer version of Ranger instead of {{RangerConfiguration}} as
> is done today (
> [https://github.com/apache/impala/blob/master/fe/src/main/java/org/apache/impala/authorization/ranger/RangerAuthorizationConfig.java#L38]).
> The second step requires us to bump up {{CDP_BUILD_NUMBER}} to 1626038 or a
> later number. Due to compatibility issues of Impala with other components in
> the toolchain, for now we just perform the first step. The second step will
> be implemented in a follow-up JIRA after we can be sure there will not be any
> compatibility issues after bumping up {{CDP_BUILD_NUMBER}}.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
