[FLINK-24432] RocksIteratorWrapper.seekToLast() calls the wrong RocksIterator method

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.14.0
• Fix Version/s: 1.14.3, 1.15.0
• Component/s: Runtime / State Backends
• Labels: pull-request-available
• Assignee: Victor Xu
• Reporter: Victor Xu
• Created: 30/Sep/21 21:09
• Updated: 15/Dec/21 01:44
• Resolved: 15/Oct/21 02:19

Description

The RocksIteratorWrapper is a wrapper of RocksIterator to do additional status check for all the methods. However, there's a typo that RocksIteratorWrapper.seekToLast() method calls RocksIterator's seekToFirst(), which is obviously wrong. I guess this issue wasn't found before as it was only referenced in the RocksTransformingIteratorWrapper.seekToLast() method and nowhere else.

@Override
public void seekToFirst() {
 iterator.seekToFirst();
 status();
}

@Override
public void seekToLast() {
 iterator.seekToFirst();
 status();
}

Issue Links

• fixes: FLINK-24479 Bug fix RocksIteratorWrapper#seekToLast method
• links to: GitHub Pull Request #17400
• links to: GitHub Pull Request #17475