Skip to content

Commit

Permalink
Merge pull request alteryx#196 from pwendell/master
Browse files Browse the repository at this point in the history
TimeTrackingOutputStream should pass on calls to close() and flush().

Without this fix you get a huge number of open files when running shuffles.

(cherry picked from commit f20093c)
Signed-off-by: Reynold Xin <[email protected]>
  • Loading branch information
rxin committed Nov 22, 2013
1 parent f678e10 commit d7c6a00
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,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 d7c6a00

Please sign in to comment.