You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues-all@impala.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2023/07/14 19:05:00 UTC

[jira] [Commented] (IMPALA-12283) Remove HIVE_HOME/lib/py from the PYTHONPATH

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

ASF subversion and git services commented on IMPALA-12283:
----------------------------------------------------------

Commit 0df29b4105c0e884cabde3289874b9850e3df21d in impala's branch refs/heads/master from Joe McDonnell
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=0df29b410 ]

IMPALA-12283: Remove Hive libs from PYTHONPATH

bin/set-pythonpath.sh include $HIVE_HOME/lib/py. This is a
historical thing that is no longer needed today. Impala
should not be getting Python code directly from Hive. As a
cleanup, this removes $HIVE_HOME/lib/py from the
PYTHONPATH.

Testing:
 - Ran a core job

Change-Id: I56d1ae3b1433d6240159f20da4680888b5f37357
Reviewed-on: http://gerrit.cloudera.org:8080/19689
Tested-by: Impala Public Jenkins <im...@cloudera.com>
Reviewed-by: Michael Smith <mi...@cloudera.com>


> Remove HIVE_HOME/lib/py from the PYTHONPATH
> -------------------------------------------
>
>                 Key: IMPALA-12283
>                 URL: https://issues.apache.org/jira/browse/IMPALA-12283
>             Project: IMPALA
>          Issue Type: Task
>          Components: Infrastructure
>    Affects Versions: Impala 4.3.0
>            Reporter: Joe McDonnell
>            Priority: Minor
>             Fix For: Impala 4.3.0
>
>
> bin/set-pythonpath.sh, there is this code:
>  
> {noformat}
> # Add Hive after Thrift because Hive supplies its own Thrift modules
> PYTHONPATH=${PYTHONPATH}:${HIVE_HOME}/lib/py{noformat}
> This is very old and we don't seem to need it. We shouldn't be getting python modules from Hive. A core job with these lines removed succeeded.
> Let's clean this up and remove this old code.
>  



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscribe@impala.apache.org
For additional commands, e-mail: issues-all-help@impala.apache.org