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

    https://github.com/apache/spark/pull/21658#discussion_r200310184
  
    --- Diff: core/src/main/scala/org/apache/spark/storage/BlockManager.scala 
---
    @@ -1569,7 +1569,7 @@ private[spark] object BlockManager {
     
         val blockManagers = new HashMap[BlockId, Seq[String]]
         for (i <- 0 until blockIds.length) {
    -      blockManagers(blockIds(i)) = blockLocations(i).map(_.host)
    +      blockManagers(blockIds(i)) = blockLocations(i).map(b => 
s"executor_${b.host}_${b.executorId}")
    --- End diff --
    
    blockIdsToLocations ?


---

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

Reply via email to