GitHub user ConeyLiu opened a pull request:
https://github.com/apache/spark/pull/19316
[SPARK-22097][CORE]Call serializationStream.close after we requested enough
memory
## What changes were proposed in this pull request?
Current code, we close the `serializationStream` after we unrolled the
block. However, there is a otential problem that the size of underlying vector
or stream maybe larger the memory we requested. So here, we need check it agin
carefully.
## How was this patch tested?
Existing UT.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ConeyLiu/spark putIteratorAsBytes
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/19316.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #19316
----
commit bfe162e3aad300414dcc3fe25a3d70025e1795dd
Author: Xianyang Liu <xianyang....@intel.com>
Date: 2017-09-22T03:29:39Z
close the serializationStream after check the memory request
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
