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

    https://github.com/apache/flink/pull/6333#discussion_r202519256
  
    --- 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()) {
    --- End diff --
    
    nit: the name `stateTable` is a bit confusion, since it is the 
`RegisteredState`(which might not be `StateTable`) now...


---

Reply via email to