GitHub user NicoK opened a pull request:
https://github.com/apache/flink/pull/5550
[FLINK-8734][network] fix partition bytes counting and re-enable in tests
## What is the purpose of the change
With [FLINK-8583], `SpillableSubpartition` does not update byte statistics
anymore when spilled by a `SpillableSubpartitionView`.
## Brief change log
- let `SpillableSubpartitionView` update its parent's statistics when
spilling buffers
- re-enable byte counting verifications in unit tests
## Verifying this change
This change re-added checks to the following tests:
- `PipelinedSubpartitionTest`
- `SpillableSubpartitionTest`
- `SubpartitionTestBase`
## Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): **no**
- The public API, i.e., is any changed class annotated with
`@Public(Evolving)`: **no**
- The serializers: **no**
- The runtime per-record code paths (performance sensitive): **no**
- Anything that affects deployment or recovery: JobManager (and its
components), Checkpointing, Yarn/Mesos, ZooKeeper: **no**
- The S3 file system connector: **no**
## Documentation
- Does this pull request introduce a new feature? **no**
- If yes, how is the feature documented? **not applicable**
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/NicoK/flink flink-8734
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/flink/pull/5550.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 #5550
----
commit 0bab9a053d8e1c7810c5f71cdb0c40b3f41a288a
Author: Nico Kruber <nico@...>
Date: 2018-02-20T17:05:54Z
[FLINK-8734][network] fix partition bytes counting and re-enable in tests
----
---
