[ 
https://issues.apache.org/jira/browse/IMPALA-9191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17060461#comment-17060461
 ] 

Joe McDonnell commented on IMPALA-9191:
---------------------------------------

Here are some code locations:

For python end to end tests, here is where we read the HIVE_MAJOR_VERSION from 
the environment: 

[https://github.com/apache/impala/blob/master/tests/common/environ.py#L75]

Trace it through to see how it is used in skip.py. 

For frontend tests, here is an example of using "Assume" to skip a frontend 
test: 
[https://github.com/apache/impala/blob/master/fe/src/test/java/org/apache/impala/analysis/AnalyzerTest.java#L538]

Here's where we startup sentry: 
[https://github.com/apache/impala/blob/master/testdata/bin/run-all.sh#L77-L79]

Let me know if there are other locations that would be useful.

> Provide a way to build Impala with only one of Sentry / Ranger
> --------------------------------------------------------------
>
>                 Key: IMPALA-9191
>                 URL: https://issues.apache.org/jira/browse/IMPALA-9191
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Frontend
>    Affects Versions: Impala 3.4.0
>            Reporter: Joe McDonnell
>            Assignee: Fang-Yu Rao
>            Priority: Critical
>
> Deployments of Impala will use either Ranger or Sentry, and deployments would 
> not switch back and forth between the two. It makes sense to provide a way to 
> pick at compile time which one to include. This allows packagers of Impala to 
> avoid a dependency for whichever authorization provider they don't need.
> In particular, compilation of the USE_CDP_HIVE=true side of Impala currently 
> needs only a few things from the CDH_BUILD_NUMBER and one them is Sentry. In 
> the other direction, the only thing a USE_CDP_HIVE=false configuration uses 
> from the CDP_BUILD_NUMBER is Ranger.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to