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 21:03:53 GMT
Alex Behm has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8198
)

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
Reviewed-on: http://gerrit.cloudera.org:8080/8198
Reviewed-by: Bharath Vissapragada <bharathv@cloudera.com>
Reviewed-by: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Reviewed-by: Alex Behm <alex.behm@cloudera.com>
Tested-by: Alex Behm <alex.behm@cloudera.com>
---
M fe/pom.xml
M fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java
2 files changed, 10 insertions(+), 11 deletions(-)

Approvals:
  Bharath Vissapragada: Looks good to me, but someone else must approve
  Dimitris Tsirogiannis: Looks good to me, approved
  Alex Behm: Looks good to me, approved; Verified

-- 
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: merged
Gerrit-Change-Id: Iddc5da8849d5aa7317d3dc572884d05dee859bdd
Gerrit-Change-Number: 8198
Gerrit-PatchSet: 5
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