Github user ArtRand commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19374#discussion_r144682554
  
    --- Diff: 
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosClusterScheduler.scala
 ---
    @@ -374,6 +375,15 @@ private[spark] class MesosClusterScheduler(
         s"${frameworkId}-${desc.submissionId}${retries}"
       }
     
    +  private def getDriverTaskId(desc: MesosDriverDescription): String = {
    +    val sId = desc.submissionId
    +    desc.retryState.map(state => sId + 
s"-retry-${state.retries.toString}").getOrElse(sId)
    +  }
    +
    +  private def getSumbmissionIdFromTaskId(taskId: String): String = {
    +    taskId.split("-retry-").head
    --- End diff --
    
    Maybe make this a constant? 


---

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

Reply via email to