flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sihuazhou <...@git.apache.org>
Subject [GitHub] flink pull request #6333: [FLINK-9489] Checkpoint timers as part of managed ...
Date Sat, 14 Jul 2018 16:26:37 GMT
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...


---

Mime
View raw message