This is an automated email from the ASF dual-hosted git repository.

dongjoon pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git


The following commit(s) were added to refs/heads/master by this push:
     new 624d3b7  [SPARK-37715][SQL][BUILD][TESTS] Remove `ojdbc6` dependency 
and update docker-integration test docs
624d3b7 is described below

commit 624d3b7c49abbf6f7bd82fa519b9270e56c660ee
Author: Danny Guinther <dguint...@seismic.com>
AuthorDate: Mon Dec 27 22:39:50 2021 -0800

    [SPARK-37715][SQL][BUILD][TESTS] Remove `ojdbc6` dependency and update 
docker-integration test docs
    
    ### What changes were proposed in this pull request?
    
    This PR aims to remove unused `ojdbc6` dependency because we are using 
`ojdbc8` in `pom.xml`.
    In addition, this updates two image names in documentation.
    
    ### Does this PR introduce _any_ user-facing change?
    No
    
    ### How was this patch tested?
    
    Pass the GitHub Action Docker test.
    
    Closes #34979 from tdg5/just-the-docker-stuff.
    
    Authored-by: Danny Guinther <dguint...@seismic.com>
    Signed-off-by: Dongjoon Hyun <dongj...@apache.org>
---
 .../scala/org/apache/spark/sql/jdbc/MsSqlServerIntegrationSuite.scala  | 3 ++-
 .../org/apache/spark/sql/jdbc/v2/MsSqlServerIntegrationSuite.scala     | 3 ++-
 project/SparkBuild.scala                                               | 3 +--
 3 files changed, 5 insertions(+), 4 deletions(-)

diff --git 
a/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/MsSqlServerIntegrationSuite.scala
 
b/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/MsSqlServerIntegrationSuite.scala
index 78015d8..5992253 100644
--- 
a/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/MsSqlServerIntegrationSuite.scala
+++ 
b/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/MsSqlServerIntegrationSuite.scala
@@ -28,7 +28,8 @@ import org.apache.spark.tags.DockerTest
 /**
  * To run this test suite for a specific version (e.g., 
2019-CU13-ubuntu-20.04):
  * {{{
- *   ENABLE_DOCKER_INTEGRATION_TESTS=1 
MSSQLSERVER_DOCKER_IMAGE_NAME=2019-CU13-ubuntu-20.04
+ *   ENABLE_DOCKER_INTEGRATION_TESTS=1
+ *   
MSSQLSERVER_DOCKER_IMAGE_NAME=mcr.microsoft.com/mssql/server:2019-CU13-ubuntu-20.04
  *     ./build/sbt -Pdocker-integration-tests
  *     "testOnly org.apache.spark.sql.jdbc.MsSqlServerIntegrationSuite"
  * }}}
diff --git 
a/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MsSqlServerIntegrationSuite.scala
 
b/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MsSqlServerIntegrationSuite.scala
index 77b7742..75446fb 100644
--- 
a/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MsSqlServerIntegrationSuite.scala
+++ 
b/external/docker-integration-tests/src/test/scala/org/apache/spark/sql/jdbc/v2/MsSqlServerIntegrationSuite.scala
@@ -31,7 +31,8 @@ import org.apache.spark.tags.DockerTest
 /**
  * To run this test suite for a specific version (e.g., 
2019-CU13-ubuntu-20.04):
  * {{{
- *   ENABLE_DOCKER_INTEGRATION_TESTS=1 
MSSQLSERVER_DOCKER_IMAGE_NAME=2019-CU13-ubuntu-20.04
+ *   ENABLE_DOCKER_INTEGRATION_TESTS=1
+ *   
MSSQLSERVER_DOCKER_IMAGE_NAME=mcr.microsoft.com/mssql/server:2019-CU13-ubuntu-20.04
  *     ./build/sbt -Pdocker-integration-tests "testOnly 
*v2*MsSqlServerIntegrationSuite"
  * }}}
  */
diff --git a/project/SparkBuild.scala b/project/SparkBuild.scala
index 2f21561..579822a 100644
--- a/project/SparkBuild.scala
+++ b/project/SparkBuild.scala
@@ -599,8 +599,7 @@ object DockerIntegrationTests {
   // This serves to override the override specified in DependencyOverrides:
   lazy val settings = Seq(
     dependencyOverrides += "com.google.guava" % "guava" % "18.0",
-    resolvers += "DB2" at 
"https://app.camunda.com/nexus/content/repositories/public/";,
-    libraryDependencies += "com.oracle" % "ojdbc6" % "11.2.0.1.0" from 
"https://app.camunda.com/nexus/content/repositories/public/com/oracle/ojdbc6/11.2.0.1.0/ojdbc6-11.2.0.1.0.jar";
 // scalastyle:ignore
+    resolvers += "DB2" at 
"https://app.camunda.com/nexus/content/repositories/public/";
   )
 }
 

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to