diff options
author | Josh Rosen <joshrosen@apache.org> | 2014-08-16 00:04:55 -0700 |
---|---|---|
committer | Josh Rosen <joshrosen@apache.org> | 2014-08-16 00:04:55 -0700 |
commit | 20fcf3d0b72f3707dc1ed95d453f570fabdefd16 (patch) | |
tree | 44d1b667829ec26aa766b21b0bd2e5cd6c7e807b /dev/run-tests-jenkins | |
parent | a83c7723bf7a90dc6cd5dde98a179303b7542020 (diff) | |
download | spark-20fcf3d0b72f3707dc1ed95d453f570fabdefd16.tar.gz spark-20fcf3d0b72f3707dc1ed95d453f570fabdefd16.tar.bz2 spark-20fcf3d0b72f3707dc1ed95d453f570fabdefd16.zip |
[SPARK-2977] Ensure ShuffleManager is created before ShuffleBlockManager
This is intended to fix SPARK-2977. Before, there was an implicit ordering dependency where we needed to know the ShuffleManager implementation before creating the ShuffleBlockManager. This patch makes that dependency explicit by adding ShuffleManager to a bunch of constructors.
I think it's a little odd for BlockManager to take a ShuffleManager only to pass it to ShuffleBlockManager without using it itself; there's an opportunity to clean this up later if we sever the circular dependencies between BlockManager and other components and pass those components to BlockManager's constructor.
Author: Josh Rosen <joshrosen@apache.org>
Closes #1976 from JoshRosen/SPARK-2977 and squashes the following commits:
a9cd1e1 [Josh Rosen] [SPARK-2977] Ensure ShuffleManager is created before ShuffleBlockManager.
Diffstat (limited to 'dev/run-tests-jenkins')
0 files changed, 0 insertions, 0 deletions