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

    https://github.com/apache/spark/pull/2933#discussion_r19377652
  
    --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala 
---
    @@ -124,6 +123,10 @@ class DAGScheduler(
       /** If enabled, we may run certain actions like take() and first() 
locally. */
       private val localExecutionEnabled = 
sc.getConf.getBoolean("spark.localExecution.enabled", false)
     
    +  /** Broadcast the serialized tasks only when they are bigger than it */
    +  private val broadcastTaskMinSize =
    +    sc.getConf.getInt("spark.scheduler.broadcastTaskMinSize", 8) * 1024
    --- End diff --
    
    If user change akka frame size to a small one, the jobs also will stop 
working silently even without this patch.
    
    I think we should have good default values for these, and assume that user 
know the risk if they want to change some configs, it's not easy to make sure 
that they are consistant between all possible values for all the configs.


---
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