Github user sihuazhou commented on a diff in the pull request: https://github.com/apache/flink/pull/5239#discussion_r168776176 --- Diff: flink-contrib/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksDBKeyedStateBackend.java --- @@ -266,11 +305,16 @@ public RocksDBKeyedStateBackend( RocksIterator iterator = db.newIterator(columnInfo.f0); iterator.seekToFirst(); - Iterable<K> iterable = () -> new RocksIteratorWrapper<>(iterator, state, keySerializer, keyGroupPrefixBytes); + Iterable<K> iterable = () -> new RocksIteratorToJavaIteratorAdapter<>(iterator, state, keySerializer, keyGroupPrefixBytes); Stream<K> targetStream = StreamSupport.stream(iterable.spliterator(), false); --- End diff -- Ok, I will check whether the namespace parameter is actually required by check the other state backend (HeapKeyedStateBackend), I think their implementation should be consistent.
---