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

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

Commit 845d468ce8add3269a4dd47d46d28266eb452b09 in impala's branch 
refs/heads/master from Riza Suminto
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=845d468ce ]

IMPALA-11611: Replace THRIFT_HOME with THIFT_JAVA_HOME in fe/pom.xml

IMPALA-11369 should replace all occurrences of THRIFT_HOME with either
of THRIFT_CPP_HOME, THRIFT_JAVA_HOME, or THRIFT_PY_HOME. However, it
miss replacing THRIFT_HOME to THRIFT_JAVA_HOME in fe/pom.xml.

This reference has not cause an issue in compilation so far, but it
should be replaced with the right one. This patch does that replacement.

Testing:
- Run and pass ./buildall.sh -skiptests -notests

Change-Id: Ic39d47375353bb4d98399b5ff765d63aa331f63d
Reviewed-on: http://gerrit.cloudera.org:8080/19040
Reviewed-by: Wenzhe Zhou <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> fe/pom.xml still has reference to THRIFT_HOME
> ---------------------------------------------
>
>                 Key: IMPALA-11611
>                 URL: https://issues.apache.org/jira/browse/IMPALA-11611
>             Project: IMPALA
>          Issue Type: Bug
>          Components: Infrastructure
>    Affects Versions: Impala 4.1.0
>            Reporter: Riza Suminto
>            Assignee: Riza Suminto
>            Priority: Minor
>
> IMPALA-11369 should replace all occurrences of THRIFT_HOME with either of 
> THRIFT_CPP_HOME, THRIFT_JAVA_HOME, or THRIFT_PY_HOME.
> There is no build problem so far, but we should replace THRIFT_HOME to 
> THRIFT_JAVA_HOME in fe/pom.xml.



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

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

Reply via email to