Myasuka commented on a change in pull request #14390:
URL: https://github.com/apache/flink/pull/14390#discussion_r545082646
##########
File path:
flink-state-backends/flink-statebackend-rocksdb/src/main/java/org/apache/flink/contrib/streaming/state/RocksIteratorWrapper.java
##########
@@ -57,7 +57,7 @@ public void seekToFirst() {
@Override
public void seekToLast() {
- iterator.seekToFirst();
+ iterator.seekToLast();
Review comment:
I think you're right to fix this. The reason why this wrong usage did
not affect the correctness of state access is that we did not ever use this
`seekToLast` in Flink.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
