Skip to content

Stop user RPC on local DB too big #175

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
ZanCorDX wants to merge 14 commits into
base: main
Choose a base branch
from
Open

Stop user RPC on local DB too big #175

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
55aa648
proper-shutdown
ZanCorDX Feb 4, 2026
548e42d
polish
ZanCorDX Feb 4, 2026
80d5818
test/lint
ZanCorDX Feb 4, 2026
7d69282
lint
ZanCorDX Feb 4, 2026
dc00c4a
trying to fix tests
ZanCorDX Feb 4, 2026
acdcef3
removed println!
ZanCorDX Feb 4, 2026
62dbf47
avoid Global executor already set error log
ZanCorDX Feb 4, 2026
3871ee2
fix: IPv4/IPv6 issues + TLS SNI
mempirate Feb 5, 2026
a5311fc
min fixes
ZanCorDX Feb 5, 2026
47a4f91
done
ZanCorDX Feb 5, 2026
490d562
new metric
ZanCorDX Feb 5, 2026
0c68418
test
ZanCorDX Feb 5, 2026
4243732
Merge main
ZanCorDX Feb 5, 2026
a91ebae
ready handler
ZanCorDX Feb 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions Cargo.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ rand = "0.9.2"
derive_more = { version = "2", features = ["deref", "from", "into"] }
fdlimit = "0.3.0"
rayon = "1.11.0"
parking_lot = { version = "0.12.3" }

# serialization
serde = { version = "1", features = ["derive"] }
Expand Down
11 changes: 11 additions & 0 deletions src/cli.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -266,6 +266,16 @@ pub struct OrderflowIngressArgs {
#[clap(long = "http.enable-gzip", default_value_t = false)]
pub gzip_enabled: bool,

/// Maximum local ClickHouse backup disk size in MB above which user RPC (e.g. eth_sendBundle)
/// is rejected with disk full. Defaults to 1024 MB (1 GiB).
#[clap(
long = "disk-max-size-to-accept-user-rpc-mb",
default_value_t = 1024,
env = "DISK_MAX_SIZE_TO_ACCEPT_USER_RPC",
id = "DISK_MAX_SIZE_TO_ACCEPT_USER_RPC"
)]
pub disk_max_size_to_accept_user_rpc_mb: u64,
Comment on lines +269 to +277
Copy link

Copilot AI Feb 11, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The flag is documented as 'MB' but the runtime conversion uses 1024 * 1024 (MiB). Either update docs/flag naming to MiB (recommended for clarity) or change the conversion to use decimal MB (1_000_000) to match the current documentation. Also consider aligning the env var name with the unit (e.g., ..._MB or ..._MIB) to avoid ambiguity.

Copilot uses AI. Check for mistakes.

/// The interval in seconds to update the peer list from BuilderHub.
#[clap(
long = "peer.update-interval-s",
Expand Down Expand Up @@ -342,6 +352,7 @@ impl Default for OrderflowIngressArgs {
score_bucket_s: 4,
log_json: false,
gzip_enabled: false,
disk_max_size_to_accept_user_rpc_mb: 1024,
tcp_small_clients: NonZero::new(1).expect("non-zero"),
tcp_big_clients: 0,
io_threads: 4,
Expand Down
Loading