kerneltime commented on PR #6022: URL: https://github.com/apache/ozone/pull/6022#issuecomment-1899421218
> @kerneltime can you explain the difference in the two options? Looking at the code it seems both options end up calling `RDBStoreAbstractIterator#seek0` soon after the iterator is created. Based on the [RocksDB API](https://github.com/facebook/rocksdb/wiki/Prefix-Seek#transition-to-the-new-usage) it does not look like the underlying implementation does anything different in these two cases. This behavior was first fixed for https://issues.apache.org/jira/browse/HDDS-8289. There needs to be a clean up of TableIterators to behave the same way when in memory vs via RocksDB. My initial attempt to clean up this code did not go as cleanly as this works only when the table type is RocksDB. I am continuing to investigate how to make the behavior the same (always initialize the iterator with prefix when prefix is available). -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
