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

    https://github.com/apache/spark/pull/186#discussion_r11052405
  
    --- Diff: core/src/main/scala/org/apache/spark/scheduler/DAGScheduler.scala 
---
    @@ -130,25 +133,45 @@ class DAGScheduler(
        * cannot be scheduled until the 
[[org.apache.spark.scheduler.DAGScheduler]] is fully constructed.
        */
       def start() {
    --- End diff --
    
    @kayousterhout I don't believe that this delayed start tactic is needed 
anymore after 
https://github.com/apache/spark/commit/0b448df6ac520a7977b1eb51e8c55e33f3fd2da8#diff-6a9ff7fb74fd490a50462d45db2d5e11
 since there is no more scheduled `resubmissionTask` -- meaning that we can go 
back to creating `eventProcessActor` (and now its supervisor) in the 
`DAGScheduler` constructor, and that the `NOTE:` in the preceeding comment is 
out of date.


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

Reply via email to