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

    https://github.com/apache/spark/pull/21589#discussion_r197725870
  
    --- Diff: 
resource-managers/mesos/src/main/scala/org/apache/spark/scheduler/cluster/mesos/MesosFineGrainedSchedulerBackend.scala
 ---
    @@ -446,6 +446,8 @@ private[spark] class MesosFineGrainedSchedulerBackend(
     
       // TODO: query Mesos for number of cores
       override def defaultParallelism(): Int = 
sc.conf.getInt("spark.default.parallelism", 8)
    +  override def numCores(): Int = defaultParallelism
    +  override def numExecutors(): Int = 1
    --- End diff --
    
    I added comments to the ticket above.


---

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

Reply via email to