spark-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From gatorsmile <...@git.apache.org>
Subject [GitHub] spark pull request #20359: [SPARK-23186][SQL] Initialize DriverManager first...
Date Wed, 24 Jan 2018 16:41:39 GMT
Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20359#discussion_r163604903
  
    --- Diff: sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/DriverRegistry.scala
---
    @@ -32,6 +32,9 @@ import org.apache.spark.util.Utils
      */
     object DriverRegistry extends Logging {
     
    +  // Initialize DriverManager first to prevent potential deadlocks between DriverManager
and Driver
    +  DriverManager.getDrivers
    --- End diff --
    
    More context about this change? Based on your PR description, this is to resolve the deadlocks
among executors? How does it work after applying this change?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org


Mime
View raw message