Github user aljoscha commented on a diff in the pull request:
https://github.com/apache/flink/pull/5230#discussion_r165385868
--- Diff:
flink-runtime/src/main/java/org/apache/flink/runtime/state/DefaultOperatorStateBackend.java
---
@@ -148,21 +170,27 @@ public void close() throws IOException {
@Override
public void dispose() {
IOUtils.closeQuietly(closeStreamOnCancelRegistry);
- registeredStates.clear();
+ registeredOperatorStates.clear();
+ registeredBroadcastStates.clear();
}
//
-------------------------------------------------------------------------------------------
// State access methods
//
-------------------------------------------------------------------------------------------
+ @Override
+ public <K, V> ReadWriteBroadcastState<K, V>
getBroadcastState(MapStateDescriptor<K, V> stateDescriptor) throws Exception {
--- End diff --
I think we don't need this because the other `getBroadcastState()` is only
ever called with `BROADCAST` mode.
---
