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

    https://github.com/apache/spark/pull/19338#discussion_r141235079
  
    --- Diff: 
core/src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala ---
    @@ -670,9 +670,12 @@ 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.*.")
    +            abort(s"""
    +              |Aborting $taskSet because task $indexInTaskSet (partition 
$partition)
    +              |cannot run anywhere due to node and executor blacklist.
    +              |Most recent failure:
    +              |${taskSetBlacklist.getLatestFailureReason}\n
    --- End diff --
    
    Actually i tested locally with something like below:
    ```
    scala> val s = s"""
         | sss\n
         | sss"""
    scala> print(s)
    
    sss
    
    sss
    ```
    And i found that there is no need to add \n since triple quoted designed to 
avoid such character.Sorry for that,and i have updated before your newest 
comment .Thanks @jerryshao 


---

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

Reply via email to