You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Huaisi Xu (Code Review)" <ge...@cloudera.org> on 2016/03/03 18:44:40 UTC

[Impala-CR](cdh5-2.2.0_5.4.x) Clean up FE pom.xml to eliminate console spew.

Hello Internal Jenkins, Alex Behm,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: Clean up FE pom.xml to eliminate console spew.
......................................................................

Clean up FE pom.xml to eliminate console spew.

This patch makes the following changes in our pom to reduce
the build time and signficantly reduce console spew.

1. Remove jar-with-dependencies from package goal.
We have no need for creating an uber jar that contains the FE as well
as all its dependencies. Locally, we carefully construct our class path
manually (relying on copy-dependencies), and in Impala deployments
the FE jar is put together with the other dependencies, so the FE jar
does not need to be self-contained.

2. Silence copy-dependencies.
Changes the configuration of the maven-dependency-plugin to not
log every copied file to the console.

Change-Id: If351e4e800fd1ca1108f9a0f4d88f52a53fc211c
Reviewed-on: http://gerrit.cloudera.org:8080/378
Reviewed-by: Alex Behm <al...@cloudera.com>
Tested-by: Internal Jenkins
(cherry picked from commit 658bd4c36d29f2eaac1f55f7ca2aadf5a533ea65)
---
M bin/impala-config.sh
M fe/pom.xml
M testdata/bin/load-hive-builtins.sh
A thirdparty/postgresql-jdbc/postgresql-9.0-801.jdbc4.jar
4 files changed, 19 insertions(+), 28 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/37/2437/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2437
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If351e4e800fd1ca1108f9a0f4d88f52a53fc211c
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.2.0_5.4.x
Gerrit-Owner: Huaisi Xu <hx...@cloudera.com>
Gerrit-Reviewer: Alex Behm <al...@cloudera.com>
Gerrit-Reviewer: Internal Jenkins