GitHub user tillrohrmann opened a pull request:
https://github.com/apache/flink/pull/1324
[FLINK-2964] [runtime] Fix broken spilling of MutableHashTable
The `HashPartition` did not count properly the number of occupied memory
segments, because it excluded the memory segments of the `BuildSideBuffer`.
That caused the `MutableHashTable` to fail when trying to spill a partition
which did not have any overflow segments. This PR fixes the problem by also
counting the memory segments of the `BuildSideBuffer`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/tillrohrmann/flink fixMutableHashTable
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/1324.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 #1324
----
commit 9db81acd4012cae2a2fc845bc762ab7082463ba2
Author: Till Rohrmann <[email protected]>
Date: 2015-11-04T01:17:12Z
[FLINK-2964] [runtime] Fix broken spilling of MutableHashTable
The HashPartition did not count properly the number of occupied memory
segments, because it excluded the memory segments of the BuildSideBuffer. That
caused the MutableHashTable to fail when trying to spill a partition which did
not have any overflow segments. This PR fixes the problem by also counting the
memory segments of the BuildSideBuffer.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
