You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/03/14 19:10:13 UTC

[GitHub] [spark] sujith71955 commented on a change in pull request #24036: [SPARK-27036][SPARK-SQL] Cancel the running jobs in the background if broadcast future timeout error occurs

sujith71955 commented on a change in pull request #24036: [SPARK-27036][SPARK-SQL] Cancel the running jobs in the background if broadcast  future timeout error occurs
URL: https://github.com/apache/spark/pull/24036#discussion_r265725915
 
 

 ##########
 File path: sql/core/src/main/scala/org/apache/spark/sql/execution/exchange/BroadcastExchangeExec.scala
 ##########
 @@ -147,6 +149,15 @@ case class BroadcastExchangeExec(
     } catch {
       case ex: TimeoutException =>
         logError(s"Could not execute broadcast in ${timeout.toSeconds} secs.", ex)
+        val executionUIData = sqlContext.sparkSession.sharedState.statusStore.
 
 Review comment:
   SQLAppStatusListener will hold the live execution data, so i use the same for getting the associated   jobs , If its not efficient way of getting then i will revisit the code and  try to find a better mechanism for getting the associated job for the particular execution Id. please let me know for any suggestions. thanks for your  valuable time.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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