[GitHub] [bookkeeper] Nicklee007 opened a new pull request, #3860: Modify incorrect rocksDB config level_compaction_dynamic_level_bytes to CFOptions

[via GitHub]

Nicklee007 opened a new pull request, #3860:
URL: https://github.com/apache/bookkeeper/pull/3860

   ### Motivation
   After PR #3056 , Bookkeeper set `level_compaction_dynamic_level_bytes=true` 
as `TableOptions` in `entry_location_rocksdb.conf.default` , which will cause 
`level_compaction_dynamic_level_bytes` lose efficacy and will cause rocksDB 
.sst file compact sort chaos when update bookie release.
   As RocksDB  conf, `level_compaction_dynamic_level_bytes` need set as 
`CFOptions` 
https://github.com/facebook/rocksdb/blob/master/examples/rocksdb_option_file_example.ini
   
   <img width="703" alt="image" 
src="https://user-images.githubusercontent.com/84127069/224640399-d5481fe5-7b75-4229-ac06-3d280aa9ae6d.png";>
   
   
   <img width="240" alt="image" 
src="https://user-images.githubusercontent.com/84127069/224640621-737d0a42-4e01-4f38-bd5a-862a93bc4b32.png";>
   
   ### Changes
   
   1. Change `level_compaction_dynamic_level_bytes=true` from `TableOptions` to 
`CFOptions`  in `entry_location_rocksdb.conf.default` ;
   
   


-- 
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: commits-unsubscr...@bookkeeper.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org