You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spark.apache.org by rx...@apache.org on 2013/11/05 05:47:22 UTC

[2/2] git commit: Merge pull request #139 from aarondav/shuffle-next

Merge pull request #139 from aarondav/shuffle-next

Never store shuffle blocks in BlockManager

After the BlockId refactor (PR #114), it became very clear that ShuffleBlocks are of no use
within BlockManager (they had a no-arg constructor!). This patch completely eliminates
them, saving us around 100-150 bytes per shuffle block.
The total, system-wide overhead per shuffle block is now a flat 8 bytes, excluding
state saved by the MapOutputTracker.

Note: This should *not* be merged directly into 0.8.0 -- see #138


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

Branch: refs/heads/master
Commit: 81065321c073fa35028b8aa110704385cba6d5b2
Parents: 0b26a39 93c9084
Author: Reynold Xin <rx...@apache.org>
Authored: Mon Nov 4 20:47:14 2013 -0800
Committer: Reynold Xin <rx...@apache.org>
Committed: Mon Nov 4 20:47:14 2013 -0800

----------------------------------------------------------------------
 .../org/apache/spark/storage/BlockInfo.scala      | 18 +-----------------
 .../org/apache/spark/storage/BlockManager.scala   | 10 ++--------
 .../apache/spark/storage/BlockObjectWriter.scala  | 12 +-----------
 3 files changed, 4 insertions(+), 36 deletions(-)
----------------------------------------------------------------------