Skip to content
Snippets Groups Projects
Commit 85a344b4 authored by Patrick Wendell's avatar Patrick Wendell
Browse files

Merge pull request #127 from kayousterhout/consolidate_schedulers

Deduplicate Local and Cluster schedulers.

The code in LocalScheduler/LocalTaskSetManager was nearly identical
to the code in ClusterScheduler/ClusterTaskSetManager. The redundancy
made making updating the schedulers unnecessarily painful and error-
prone. This commit combines the two into a single TaskScheduler/
TaskSetManager.

Unfortunately the diff makes this change look much more invasive than it is -- TaskScheduler.scala is only superficially changed (names updated, overrides removed) from the old ClusterScheduler.scala, and the same with
TaskSetManager.scala.

Thanks @rxin for suggesting this change!
parents c2dd6bcd 1efe3adf
No related branches found
No related tags found
No related merge requests found
Showing
with 908 additions and 1246 deletions
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment