From 8b69b17f3fde2c890068a5a9ef490fe675cc80c1 Mon Sep 17 00:00:00 2001 From: zhoukang Date: Mon, 7 Aug 2017 12:51:39 +0100 Subject: [PATCH] [SPARK-21544][DEPLOY][TEST-MAVEN] Tests jar of some module should not upload twice ## What changes were proposed in this pull request? **For moudle below:** common/network-common streaming sql/core sql/catalyst **tests.jar will install or deploy twice.Like:** `[DEBUG] Installing org.apache.spark:spark-streaming_2.11/maven-metadata.xml to /home/mi/.m2/repository/org/apache/spark/spark-streaming_2.11/maven-metadata-local.xml [INFO] Installing /home/mi/Work/Spark/scala2.11/spark/streaming/target/spark-streaming_2.11-2.1.0-mdh2.1.0.1-SNAPSHOT-tests.jar to /home/mi/.m2/repository/org/apache/spark/spark-streaming_2.11/2.1.0-mdh2.1.0.1-SNAPSHOT/spark-streaming_2.11-2.1.0-mdh2.1.0.1-SNAPSHOT-tests.jar [DEBUG] Skipped re-installing /home/mi/Work/Spark/scala2.11/spark/streaming/target/spark-streaming_2.11-2.1.0-mdh2.1.0.1-SNAPSHOT-tests.jar to /home/mi/.m2/repository/org/apache/spark/spark-streaming_2.11/2.1.0-mdh2.1.0.1-SNAPSHOT/spark-streaming_2.11-2.1.0-mdh2.1.0.1-SNAPSHOT-tests.jar, seems unchanged` **The reason is below:** `[DEBUG] (f) artifact = org.apache.spark:spark-streaming_2.11:jar:2.1.0-mdh2.1.0.1-SNAPSHOT [DEBUG] (f) attachedArtifacts = [org.apache.spark:spark-streaming_2.11:test-jar:tests:2.1.0-mdh2.1.0.1-SNAPSHOT, org.apache.spark:spark-streaming_2.11:jar:tests:2.1.0-mdh2.1.0.1-SNAPSHOT, org.apache.spark:spark -streaming_2.11:java-source:sources:2.1.0-mdh2.1.0.1-SNAPSHOT, org.apache.spark:spark-streaming_2.11:java-source:test-sources:2.1.0-mdh2.1.0.1-SNAPSHOT, org.apache.spark:spark-streaming_2.11:javadoc:javadoc:2.1.0 -mdh2.1.0.1-SNAPSHOT]` when executing 'mvn deploy' to nexus during release.I will fail since release nexus can not be overrided. ## How was this patch tested? Execute 'mvn clean install -Pyarn -Phadoop-2.6 -Phadoop-provided -DskipTests' Author: zhoukang Closes #18745 from caneGuy/zhoukang/fix-installtwice. --- common/network-common/pom.xml | 2 +- sql/catalyst/pom.xml | 7 +------ sql/core/pom.xml | 7 +------ 3 files changed, 3 insertions(+), 13 deletions(-) diff --git a/common/network-common/pom.xml b/common/network-common/pom.xml index 0254d0cefc..ccd8504ba0 100644 --- a/common/network-common/pom.xml +++ b/common/network-common/pom.xml @@ -126,7 +126,7 @@ maven-jar-plugin - test-jar-on-test-compile + prepare-test-jar test-compile test-jar diff --git a/sql/catalyst/pom.xml b/sql/catalyst/pom.xml index 0bbf7a9512..fce8149379 100644 --- a/sql/catalyst/pom.xml +++ b/sql/catalyst/pom.xml @@ -118,12 +118,7 @@ maven-jar-plugin - - test-jar - - - - test-jar-on-test-compile + prepare-test-jar test-compile test-jar diff --git a/sql/core/pom.xml b/sql/core/pom.xml index 661c31ded7..25004e5326 100644 --- a/sql/core/pom.xml +++ b/sql/core/pom.xml @@ -174,12 +174,7 @@ maven-jar-plugin - - test-jar - - - - test-jar-on-test-compile + prepare-test-jar test-compile test-jar