Github user StefanRRichter commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6333#discussion_r202556091
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/state/heap/HeapKeyedStateBackend.java
 ---
    @@ -446,8 +485,10 @@ public String toString() {
        @Override
        public int numStateEntries() {
                int sum = 0;
    -           for (StateTable<K, ?, ?> stateTable : stateTables.values()) {
    -                   sum += stateTable.size();
    +           for (StateSnapshotRestore stateTable : 
registeredStates.values()) {
    +                   if (stateTable instanceof StateTable) {
    +                           sum += ((StateTable<?, ?, ?>) 
stateTable).size();
    +                   }
    --- End diff --
    
    This method is only used for some tests, and to be on the safe side it 
probably only expected to count the keyed state and not some timers.


---

Reply via email to