Skip to content

Commit

Permalink
TimeTrackingOutputStream should pass on calls to close() and flush().
Browse files Browse the repository at this point in the history
Without this fix you get a huge number of open shuffles after running
shuffles.
  • Loading branch information
pwendell committed Nov 22, 2013
1 parent 4ba3267 commit 53b94ef
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,8 @@ class DiskBlockObjectWriter(
def write(i: Int): Unit = callWithTiming(out.write(i))
override def write(b: Array[Byte]) = callWithTiming(out.write(b))
override def write(b: Array[Byte], off: Int, len: Int) = callWithTiming(out.write(b, off, len))
override def close() = out.close()
override def flush() = out.flush()
}

private val syncWrites = System.getProperty("spark.shuffle.sync", "false").toBoolean
Expand Down

0 comments on commit 53b94ef

Please sign in to comment.