spark git commit: [SPARK-17473][SQL] fixing docker integration tests error due to different versions of jars.
Repository: spark Updated Branches: refs/heads/branch-2.0 c4660d607 -> f56035ba6 [SPARK-17473][SQL] fixing docker integration tests error due to different versions of jars. ## What changes were proposed in this pull request? Docker tests are using older version of jersey jars (1.19), which was used in older releases of spark. In 2.0 releases Spark was upgraded to use 2.x verison of Jersey. After upgrade to new versions, docker tests are failing with AbstractMethodError. Now that spark is upgraded to 2.x jersey version, using of shaded docker jars may not be required any more. Removed the exclusions/overrides of jersey related classes from pom file, and changed the docker-client to use regular jar instead of shaded one. ## How was this patch tested? Tested using existing docker-integration-tests Author: sureshthalamatiCloses #15114 from sureshthalamati/docker_testfix-spark-17473. (cherry picked from commit cdea1d1343d02f0077e1f3c92ca46d04a3d30414) Signed-off-by: Josh Rosen Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/f56035ba Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/f56035ba Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/f56035ba Branch: refs/heads/branch-2.0 Commit: f56035ba6c86fe93a45fd437f98f812431df0069 Parents: c4660d6 Author: sureshthalamati Authored: Mon Sep 19 09:56:16 2016 -0700 Committer: Josh Rosen Committed: Mon Sep 19 10:29:57 2016 -0700 -- external/docker-integration-tests/pom.xml | 68 -- pom.xml | 1 - 2 files changed, 69 deletions(-) -- http://git-wip-us.apache.org/repos/asf/spark/blob/f56035ba/external/docker-integration-tests/pom.xml -- diff --git a/external/docker-integration-tests/pom.xml b/external/docker-integration-tests/pom.xml index 18e14c7..0ca94e5 100644 --- a/external/docker-integration-tests/pom.xml +++ b/external/docker-integration-tests/pom.xml @@ -49,38 +49,7 @@ com.spotify docker-client - shaded test - - - - com.fasterxml.jackson.jaxrs - jackson-jaxrs-json-provider - - - com.fasterxml.jackson.datatype - jackson-datatype-guava - - - com.fasterxml.jackson.core - jackson-databind - - - org.glassfish.jersey.core - jersey-client - - - org.glassfish.jersey.connectors - jersey-apache-connector - - - org.glassfish.jersey.media - jersey-media-json-jackson - - org.apache.httpcomponents @@ -152,43 +121,6 @@ test - - - com.sun.jersey - jersey-server - 1.19 - test - - - com.sun.jersey - jersey-core - 1.19 - test - - - com.sun.jersey - jersey-servlet - 1.19 - test - - - com.sun.jersey - jersey-json - 1.19 - test - - - stax - stax-api - - - - -
spark git commit: [SPARK-17473][SQL] fixing docker integration tests error due to different versions of jars.
Repository: spark Updated Branches: refs/heads/master d720a4019 -> cdea1d134 [SPARK-17473][SQL] fixing docker integration tests error due to different versions of jars. ## What changes were proposed in this pull request? Docker tests are using older version of jersey jars (1.19), which was used in older releases of spark. In 2.0 releases Spark was upgraded to use 2.x verison of Jersey. After upgrade to new versions, docker tests are failing with AbstractMethodError. Now that spark is upgraded to 2.x jersey version, using of shaded docker jars may not be required any more. Removed the exclusions/overrides of jersey related classes from pom file, and changed the docker-client to use regular jar instead of shaded one. ## How was this patch tested? Tested using existing docker-integration-tests Author: sureshthalamatiCloses #15114 from sureshthalamati/docker_testfix-spark-17473. Project: http://git-wip-us.apache.org/repos/asf/spark/repo Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/cdea1d13 Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/cdea1d13 Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/cdea1d13 Branch: refs/heads/master Commit: cdea1d1343d02f0077e1f3c92ca46d04a3d30414 Parents: d720a40 Author: sureshthalamati Authored: Mon Sep 19 09:56:16 2016 -0700 Committer: Josh Rosen Committed: Mon Sep 19 09:56:16 2016 -0700 -- external/docker-integration-tests/pom.xml | 68 -- pom.xml | 1 - 2 files changed, 69 deletions(-) -- http://git-wip-us.apache.org/repos/asf/spark/blob/cdea1d13/external/docker-integration-tests/pom.xml -- diff --git a/external/docker-integration-tests/pom.xml b/external/docker-integration-tests/pom.xml index 7417199..57d553b 100644 --- a/external/docker-integration-tests/pom.xml +++ b/external/docker-integration-tests/pom.xml @@ -49,38 +49,7 @@ com.spotify docker-client - shaded test - - - - com.fasterxml.jackson.jaxrs - jackson-jaxrs-json-provider - - - com.fasterxml.jackson.datatype - jackson-datatype-guava - - - com.fasterxml.jackson.core - jackson-databind - - - org.glassfish.jersey.core - jersey-client - - - org.glassfish.jersey.connectors - jersey-apache-connector - - - org.glassfish.jersey.media - jersey-media-json-jackson - - org.apache.httpcomponents @@ -152,43 +121,6 @@ test - - - com.sun.jersey - jersey-server - 1.19 - test - - - com.sun.jersey - jersey-core - 1.19 - test - - - com.sun.jersey - jersey-servlet - 1.19 - test - - - com.sun.jersey - jersey-json - 1.19 - test - - - stax - stax-api - - - - -