LuciferYang opened a new pull request, #36200: URL: https://github.com/apache/spark/pull/36200
### What changes were proposed in this pull request? `ExternalShuffleBlockResolver` and `YarnShuffleService` directly use `LevelDB`, this is not conducive to extending the use of `RocksDB` in this scenario. This pr is encapsulated for expansibility. It will be the pre pr of SPARK-38888, the main change as follows: - Introduce `LocalDB` interface and implementation of corresponding `LevelDBImpl` now, `RocksDB` implementation will be involved in the future - Introduce `LocalDBProvider` to initialization `LocalDB`, it is only used to generate `LevelDBImpl` now, `RocksDB` implementation will be involved in the future - Change `StoreVersion` to top-level class, it will be reused by `RocksDB` implementation - Replace the directly use of `LevelDB` in `ExternalShuffleBlockResolver` and `YarnShuffleService` with `LocalDB` - fix corresponding UTs and test tools ### Why are the changes needed? This is pre pr of SPARK-38888 ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Pass GA -- 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]
