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

    https://github.com/apache/spark/pull/19338#discussion_r141226978
  
    --- Diff: 
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
    @@ -671,8 +671,10 @@ private[spark] class TaskSetManager(
               if (blacklistedEverywhere) {
                 val partition = tasks(indexInTaskSet).partitionId
                 abort(s"Aborting $taskSet because task $indexInTaskSet 
(partition $partition) " +
    -              s"cannot run anywhere due to node and executor blacklist.  
Blacklisting behavior " +
    -              s"can be configured via spark.blacklist.*.")
    +              s"cannot run anywhere due to node and executor blacklist.\n" 
+
    --- End diff --
    
    Can we change to Scala's triple quoted string interpolation here? you can 
refer to 
[here](https://github.com/apache/spark/blob/ceaec93839d18a20e0cd78b70f3ea71872dce0a4/core/src/main/scala/org/apache/spark/api/python/SerDeUtil.scala#L211)


---

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

Reply via email to