From c55a6e55a10275a2a702042842372cf169890bbc Mon Sep 17 00:00:00 2001 From: Trent Nelson Date: Tue, 14 Nov 2023 15:56:47 -0700 Subject: [PATCH] validator: deprecates `--disable-accounts-disk-index` (#33850) * validator: deprecates `--disable-accounts-disk-index` * Fix checks failure * Move disable_accounts_disk_index arg to alphabetical order --------- Co-authored-by: Will Hickey --- validator/src/cli.rs | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/validator/src/cli.rs b/validator/src/cli.rs index 9aa1c466f8e336..364195e3083cd1 100644 --- a/validator/src/cli.rs +++ b/validator/src/cli.rs @@ -1246,12 +1246,6 @@ pub fn app<'a>(version: &'a str, default_args: &'a DefaultArgs) -> App<'a, 'a> { .takes_value(true) .help("How much memory the accounts index can consume. If this is exceeded, some account index entries will be stored on disk."), ) - .arg( - Arg::with_name("disable_accounts_disk_index") - .long("disable-accounts-disk-index") - .help("Disable the disk-based accounts index if it is enabled by default.") - .conflicts_with("accounts_index_memory_limit_mb") - ) .arg( Arg::with_name("accounts_index_bins") .long("accounts-index-bins") @@ -1793,6 +1787,10 @@ fn deprecated_arguments() -> Vec { Ok(()) } })); + add_arg!(Arg::with_name("disable_accounts_disk_index") + .long("disable-accounts-disk-index") + .help("Disable the disk-based accounts index if it is enabled by default.") + .conflicts_with("accounts_index_memory_limit_mb")); add_arg!( Arg::with_name("disable_quic_servers") .long("disable-quic-servers")