Github user NicoK commented on a diff in the pull request:
https://github.com/apache/flink/pull/4559#discussion_r153564062
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/io/network/partition/SpillableSubpartitionView.java
---
@@ -133,7 +135,7 @@ int releaseMemory() throws IOException {
}
@Override
- public Buffer getNextBuffer() throws IOException, InterruptedException {
+ public Buffer getNextBufferInternal() throws IOException,
InterruptedException {
--- End diff --
make this `protected`
---