Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/6156#discussion_r195116411
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/heap/AbstractHeapMergingState.java
---
@@ -34,12 +33,10 @@
* @param <IN> The type of the input elements.
* @param <SV> The type of the values in the state.
* @param <OUT> The type of the output elements.
- * @param <S> The type of State
*/
-public abstract class AbstractHeapMergingState<K, N, IN, SV, OUT, S
extends State>
- extends AbstractHeapState<K, N, SV, S>
- implements InternalMergingState<K, N, IN, SV, OUT> {
-
+public abstract class AbstractHeapMergingState<K, N, IN, SV, OUT>
+ extends AbstractHeapState<K, N, SV>
+ implements InternalMergingState<K, N, IN, SV, OUT>,
org.apache.flink.runtime.state.internal.InternalAppendingState<K, N, IN, SV,
OUT> {
--- End diff --
I would make this a regular import.
---