Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5885#discussion_r184086555
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/heap/HeapAggregatingState.java
---
@@ -47,21 +47,23 @@
/**
* Creates a new key/value state for the given hash map of key/value
pairs.
*
- * @param stateDesc
- * The state identifier for the state. This contains name
and can create a default state value.
+ * @param valueSerializer
+ * The serializer for the state.
* @param stateTable
* The state table to use in this kev/value state. May
contain initial state.
* @param namespaceSerializer
* The serializer for the type that indicates the namespace
*/
public HeapAggregatingState(
- AggregatingStateDescriptor<IN, ACC, OUT> stateDesc,
StateTable<K, N, ACC> stateTable,
TypeSerializer<K> keySerializer,
- TypeSerializer<N> namespaceSerializer) {
+ TypeSerializer<ACC> valueSerializer,
+ TypeSerializer<N> namespaceSerializer,
+ ACC defaultValue,
--- End diff --
The comments are not updated to reflect this and the next line.
---
