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 2014/01/22 23:06:15 UTC
[2/2] git commit: Merge pull request #493 from
kayousterhout/double_add
Merge pull request #493 from kayousterhout/double_add
Fixed bug where task set managers are added to queue twice
@mateiz can you verify that this is a bug and wasn't intentional? (https://github.com/apache/incubator-spark/commit/90a04dab8d9a2a9a372cea7cdf46cc0fd0f2f76c#diff-7fa4f84a961750c374f2120ca70e96edR551)
This bug leads to a small performance hit because task
set managers will get offered each rejected resource
offer twice, but doesn't lead to any incorrect functionality.
Thanks to @hdc1112 for pointing this out.
Project: http://git-wip-us.apache.org/repos/asf/incubator-spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-spark/commit/5bcfd798
Tree: http://git-wip-us.apache.org/repos/asf/incubator-spark/tree/5bcfd798
Diff: http://git-wip-us.apache.org/repos/asf/incubator-spark/diff/5bcfd798
Branch: refs/heads/master
Commit: 5bcfd798117e8617e604c1dd3b5c9b67e83100bb
Parents: d009b17 19da82c
Author: Matei Zaharia <ma...@databricks.com>
Authored: Wed Jan 22 14:05:48 2014 -0800
Committer: Matei Zaharia <ma...@databricks.com>
Committed: Wed Jan 22 14:05:48 2014 -0800
----------------------------------------------------------------------
.../src/main/scala/org/apache/spark/scheduler/TaskSetManager.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------