lhotari opened a new pull request, #583:
URL: https://github.com/apache/pulsar-helm-chart/pull/583

   ### Motivation
   
   Follow-up on #580
   
   Pulsar has provided individual RocksDB config files to configure 
BookKeeper's RocksDB databases.
   However, this configuration hasn't been consistently used since Pulsar's 
bookkeeper.conf doesn't contain `defaultRocksdbConf`, 
`entryLocationRocksdbConf` and `ledgerMetadataRocksdbConf` files.
   
   Related BookKeeper changes:
   - https://github.com/apache/bookkeeper/pull/3056 in 4.15.0
   - https://github.com/apache/bookkeeper/pull/3523 in 4.15.3 / 4.16.0
   
   Related Pulsar changes:
   - https://github.com/apache/pulsar/pull/15142
   - https://github.com/apache/pulsar/pull/21157
   - https://github.com/apache/pulsar/pull/23175
   - https://github.com/apache/pulsar/pull/23178
   
   ### Modifications
   
   - enable configuration so that the RockDB config files really get used.
   - use `bin/update-rocksdb-conf-from-env.py` script to apply 
`dbStorage_rocksDB_*` environment variables to update the config files
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   


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