impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Philip Zeyliger (Code Review)" <>
Subject [Impala-ASF-CR] Remove unused deps, centralize some pom versions, upgrade SLF4J and commons-io.
Date Tue, 19 Dec 2017 18:10:56 GMT
Philip Zeyliger has uploaded this change for review. (

Change subject: Remove unused deps, centralize some pom versions, upgrade SLF4J and commons-io.

Remove unused deps, centralize some pom versions, upgrade SLF4J and commons-io.

As a follow-on to centralizing into one parent pom, we can now manage
thirdparty dependency versions in Java a little bit more clearly.

Upgrades SLF4J,
  slf4j: 1.7.5 -> 1.7.25 2.4 -> 2.6

  The SLF4J upgrade is nice to be able to run under Java9. The release
  notes at are uneventful.

  Commons IO 2.6 supports Java 9 and is source and binary compatible,
  per and

Removes the following dependencies:

  I ran "mvn dependency:analyze" and these were some (but not all)
  of the "Unused declared dependencies found." Spelunking in git logs,
  these dependencies are from 2013 and possibly from an effort
  to run with dependencies from the filesystem. They don't seem
  to be required anymore.

Stops pulling in an old version of hadoop-client and kite-data-core in
testdata/TableFlattener by using the same versions as the Hadoop we use.
Doing so was unnecessarily causing us to download extra, old Hadoop
jars, and the new Hadoop jars seem to work just as well. This is the
kind of divergence that centralizing the versions into variables will
help with.

Creates variables for:
  kite.version (controlled via $IMPALA_KITE_VERSION in

Cleans up unused IMPALA_PARQUET_URL variables in We
only download Parquet via Maven, rather than downloading it in the
toolchain, so this variable wasn't doing anything.

I ran the core tests with this change.

Change-Id: I717e0625dfe0fdbf7e9161312e9e80f405a359c5
M bin/
M fe/pom.xml
M impala-parent/pom.xml
M testdata/TableFlattener/pom.xml
M testdata/pom.xml
5 files changed, 22 insertions(+), 46 deletions(-)

  git pull ssh:// refs/changes/53/8853/2
To view, visit
To unsubscribe, visit

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I717e0625dfe0fdbf7e9161312e9e80f405a359c5
Gerrit-Change-Number: 8853
Gerrit-PatchSet: 2
Gerrit-Owner: Philip Zeyliger <>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Philip Zeyliger <>

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