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

    https://github.com/apache/spark/pull/7268#discussion_r34648286
  
    --- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
    @@ -124,10 +124,14 @@ private[spark] abstract class MapOutputTracker(conf: 
SparkConf) extends Logging
       }
     
       /**
    -   * Called from executors to get the server URIs and output sizes of the 
map outputs of
    -   * a given shuffle.
    +   * Called from executors to get the server URIs and output sizes for 
each shuffle block that
    +   * needs to be read from a given reduce task.
        */
    -  def getServerStatuses(shuffleId: Int, reduceId: Int): 
Array[(BlockManagerId, Long)] = {
    +  def getMapSizesByExecutorId(shuffleId: Int, reduceId: Int)
    --- End diff --
    
    while you are at it, can you document what the long is? (i'm guessing it is 
size of the block?)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to