JingsongLi opened a new pull request, #8886:
URL: https://github.com/apache/paimon/pull/8886

   ## What is changed
   
   - Remove the RocksDB-backed lookup state implementations and their tests.
   - Remove the `frocksdbjni` dependencies from core, Flink 1.17, and micro 
benchmarks.
   - Move the remaining lookup cache options to `CoreOptions`.
   - Remove RocksDB-specific benchmark cases and configuration documentation.
   
   ## Why
   
   Lookup state now uses `LocalKvDb`, so keeping the unused RocksDB 
implementation and native dependency increases the binary footprint and 
maintenance cost without providing a runtime path.
   
   ## User impact
   
   The obsolete `rocksdb.*` lookup tuning options are removed. Lookup cache 
sizing now consistently uses `lookup.cache-max-memory-size`; the remaining 
generic lookup options keep their existing keys.
   
   ## Validation
   
   - Compiled core, Flink common, and micro-benchmark modules with dependencies.
   - Compiled micro-benchmark test sources.
   - Ran `LocalKvStateFactoryTest` and `GlobalIndexAssignerTest` (22 tests).
   - Ran `FileStoreLookupFunctionTest` and `DynamicPartitionLevelLoaderTest` 
(18 tests).
   - Compiled Flink 1.17 test sources with the Flink 1 profile.
   - Verified dependency trees contain no RocksDB artifacts.
   - Verified no RocksDB implementation, JNI, or `rocksdb.*` configuration 
references remain.
   


-- 
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]

Reply via email to