impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Alex Behm (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-6009: Upgrade Guava to 14.0.1
Date Tue, 03 Oct 2017 18:12:58 GMT
Hello Bharath Vissapragada, Michael Brown, Dimitris Tsirogiannis, Impala Public Jenkins, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/8198

to look at the new patch set (#4).

Change subject: IMPALA-6009: Upgrade Guava to 14.0.1
......................................................................

IMPALA-6009: Upgrade Guava to 14.0.1

Builds are breaking because the latest hive-exec
snapshot jar includes Guava 14.0.1 classes which
conflict with the version Impala depends on (11.0.2).

Between versions 11.0.2 and 14.0.1 Guava changed the
API of some Hasher methods that Impala uses.

As a workaround this patch upgrades Impala's Guava
dependency to version 14.0.1 to be consistent with
the classes in hive-exec.

Testing:
- mvn compile succeeded

Change-Id: Iddc5da8849d5aa7317d3dc572884d05dee859bdd
---
M fe/pom.xml
M fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java
2 files changed, 10 insertions(+), 11 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/98/8198/4
-- 
To view, visit http://gerrit.cloudera.org:8080/8198
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iddc5da8849d5aa7317d3dc572884d05dee859bdd
Gerrit-Change-Number: 8198
Gerrit-PatchSet: 4
Gerrit-Owner: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Alex Behm <alex.behm@cloudera.com>
Gerrit-Reviewer: Bharath Vissapragada <bharathv@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Michael Brown <mikeb@cloudera.com>

Mime
  • Unnamed multipart/alternative (inline, 8-Bit, 0 bytes)
View raw message