hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hive QA (Jira)" <>
Subject [jira] [Commented] (HIVE-22126) hive-exec packaging should shade guava
Date Wed, 11 Mar 2020 11:58:00 GMT


Hive QA commented on HIVE-22126:

Here are the results of testing the latest attachment:

{color:red}ERROR:{color} -1 due to no test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 3 failed/errored test(s), 18066 tests executed
*Failed tests:*
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[schq_ingest] (batchId=185) (batchId=293)
org.apache.hive.jdbc.TestNewGetSplitsFormatReturnPath.testLlapInputFormatEndToEnd (batchId=293)

Test results:
Console output:
Test logs:

Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 3 tests failed

This message is automatically generated.

ATTACHMENT ID: 12996323 - PreCommit-HIVE-Build

> hive-exec packaging should shade guava
> --------------------------------------
>                 Key: HIVE-22126
>                 URL:
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Vihang Karajgaonkar
>            Assignee: Eugene Chung
>            Priority: Major
>             Fix For: 4.0.0
>         Attachments: HIVE-22126.01.patch, HIVE-22126.02.patch, HIVE-22126.03.patch, HIVE-22126.04.patch,
HIVE-22126.05.patch, HIVE-22126.06.patch, HIVE-22126.07.patch, HIVE-22126.08.patch, HIVE-22126.09.patch,
> The ql/pom.xml includes complete guava library into hive-exec.jar
This causes a problems for downstream clients of hive which have hive-exec.jar in their classpath
since they are pinned to the same guava version as that of hive. 
> We should shade guava classes so that other components which depend on hive-exec can
independently use a different version of guava as needed.

This message was sent by Atlassian Jira

View raw message