You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by ma...@apache.org on 2013/10/17 20:12:34 UTC

[3/3] git commit: Merge pull request #67 from kayousterhout/remove_tsl

Merge pull request #67 from kayousterhout/remove_tsl

Removed TaskSchedulerListener interface.

The interface was used only by the DAG scheduler (so it wasn't necessary
to define the additional interface), and the naming makes it very
confusing when reading the code (because "listener" was used
to describe the DAG scheduler, rather than SparkListeners, which
implement a nearly-identical interface but serve a different
function).

@mateiz - is there a reason for this interface that I'm missing?


Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/cf64f63f
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/cf64f63f
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/cf64f63f

Branch: refs/heads/master
Commit: cf64f63f8a3b54dec37e991856260ac63f7e222e
Parents: f9973ca 809f547
Author: Matei Zaharia <ma...@eecs.berkeley.edu>
Authored: Thu Oct 17 11:12:28 2013 -0700
Committer: Matei Zaharia <ma...@eecs.berkeley.edu>
Committed: Thu Oct 17 11:12:28 2013 -0700

----------------------------------------------------------------------
 .../apache/spark/scheduler/DAGScheduler.scala   | 14 +++---
 .../apache/spark/scheduler/TaskScheduler.scala  |  7 ++-
 .../spark/scheduler/TaskSchedulerListener.scala | 44 ------------------
 .../scheduler/cluster/ClusterScheduler.scala    | 14 +++---
 .../cluster/ClusterTaskSetManager.scala         | 21 +++++----
 .../spark/scheduler/local/LocalScheduler.scala  |  6 +--
 .../scheduler/local/LocalTaskSetManager.scala   | 11 +++--
 .../spark/scheduler/DAGSchedulerSuite.scala     |  2 +-
 .../cluster/ClusterTaskSetManagerSuite.scala    | 49 ++++++++++----------
 9 files changed, 63 insertions(+), 105 deletions(-)
----------------------------------------------------------------------