PokIsemaine commented on PR #2332: URL: https://github.com/apache/kvrocks/pull/2332#issuecomment-2270403804
About the latest commit https://github.com/apache/kvrocks/pull/2332/commits/822712da596effe6a3c29c4e7d6aa11fe8c648bc There was a difference in the behavior of `Speedb` and `RocksDB` that caused stream `AutoClaim` to fail in some tests. Since I'm not very familiar with `Speedb`, I didn't find exactly what caused the difference.  I referred to the related PR of `AutoClaim` and found that it has been refactored recently and tried to add the key type check. The test currently passed. https://github.com/apache/kvrocks/pull/2373 https://github.com/apache/kvrocks/pull/2384 If there may be issues with this modification, please let me know. Thank you very much. -- 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]
