anishshri-db opened a new pull request, #41410: URL: https://github.com/apache/spark/pull/41410
### What changes were proposed in this pull request? Use keyMayExist to check if key is absent and avoid gets while tracking metrics using RocksDB state store provider ### Why are the changes needed? Small change to use lighter weight check for key existence ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Ran all relevant unit tests - RocksDBSuite - RocksDBStateStoreSuite - RocksDBStateStoreIntegrationSuite ``` [info] Run completed in 41 seconds, 621 milliseconds. [info] Total number of tests run: 33 [info] Suites: completed 1, aborted 0 [info] Tests: succeeded 33, failed 0, canceled 0, ignored 0, pending 0 [info] All tests passed. [info] Run completed in 54 seconds, 675 milliseconds. [info] Total number of tests run: 73 [info] Suites: completed 1, aborted 0 [info] Tests: succeeded 73, failed 0, canceled 0, ignored 0, pending 0 [info] All tests passed. [info] Run completed in 14 seconds, 892 milliseconds. [info] Total number of tests run: 5 [info] Suites: completed 1, aborted 0 [info] Tests: succeeded 5, failed 0, canceled 0, ignored 0, pending 0 [info] All tests passed. ``` -- 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]
