Github user azagrebin commented on a diff in the pull request:
https://github.com/apache/flink/pull/6313#discussion_r202081209
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/heap/HeapKeyedStateBackend.java
---
@@ -240,7 +243,7 @@ private boolean hasRegisteredState() {
}
@Override
- public <N, SV, S extends State, IS extends S> IS createState(
+ public <N, SV, S extends State, IS extends S> IS createInternalState(
--- End diff --
The method uses state descriptor to produce backend specific
`InternalKvState`s. Is it also applicable for timers? It could be named
`createInternalKvState` and `InternalKvStateFactory`.
---
