Skip to content

Conversation

Oppen
Copy link
Contributor

@Oppen Oppen commented Oct 1, 2025

  • default options
  • fix warnings
  • feat(l1): allow overriding RocksDB config

Motivation

Description

Closes #issue_number

@github-actions github-actions bot added the L1 Ethereum client label Oct 1, 2025
Copy link

github-actions bot commented Oct 1, 2025

Lines of code report

Total lines added: 0
Total lines removed: 79
Total lines changed: 79

Detailed view
+-------------------------------------------+-------+------+
| File                                      | Lines | Diff |
+-------------------------------------------+-------+------+
| ethrex/crates/storage/store_db/rocksdb.rs | 1061  | -79  |
+-------------------------------------------+-------+------+

fedacking and others added 4 commits October 2, 2025 14:37
- (!) Patches temporarily (do not merge!) the RocksDB dep to a personal repo
  containing a feature I sent upstream that exposes a function needed
  for this feature;
- Introduces a new `ETHREX_ROCKSDB_OPTIONS_OVERRIDE` environment
  variable to allow for faster testing of different settings.

The syntax is described in the following
[blogpost](https://rocksdb.org/blog/2015/02/24/reading-rocksdb-options-from-a-file.html).
A user or developer should set this variable with the appropriate text.
For convenience, we might want to store the settings in a file. It can
be loaded by running:
```sh
export ETHREX_ROCKSDB_OPTIONS_OVERRIDE=$(cat example.conf)
```
@Oppen Oppen force-pushed the feat/rocksdb_config_file branch from ce6daa3 to 44d51d4 Compare October 2, 2025 17:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L1 Ethereum client
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants