chore: SelectorType use snake_case (#2962)

chore: let SelectorType use snake_case
This commit is contained in:
JeremyHi
2023-12-20 10:28:29 +08:00
committed by GitHub
parent 839e653e0d
commit 5f8c17514f
2 changed files with 15 additions and 9 deletions

View File

@@ -7,10 +7,10 @@ server_addr = "127.0.0.1:3002"
# Etcd server address, "127.0.0.1:2379" by default.
store_addr = "127.0.0.1:2379"
# Datanode selector type.
# - "LeaseBased" (default value).
# - "LoadBased"
# - "lease_based" (default value).
# - "load_based"
# For details, please see "https://docs.greptime.com/developer-guide/metasrv/selector".
selector = "LeaseBased"
selector = "lease_based"
# Store data in memory, false by default.
use_memory_store = false
# Whether to enable greptimedb telemetry, true by default.

View File

@@ -67,8 +67,8 @@ impl TryFrom<&str> for SelectorType {
fn try_from(value: &str) -> Result<Self> {
match value {
"LoadBased" => Ok(SelectorType::LoadBased),
"LeaseBased" => Ok(SelectorType::LeaseBased),
"load_based" | "LoadBased" => Ok(SelectorType::LoadBased),
"lease_based" | "LeaseBased" => Ok(SelectorType::LeaseBased),
other => error::UnsupportedSelectorTypeSnafu {
selector_type: other,
}
@@ -89,12 +89,18 @@ mod tests {
#[test]
fn test_convert_str_to_selector_type() {
let leasebased = "LeaseBased";
let selector_type = leasebased.try_into().unwrap();
let lease_based = "lease_based";
let selector_type = lease_based.try_into().unwrap();
assert_eq!(SelectorType::LeaseBased, selector_type);
let lease_based = "LeaseBased";
let selector_type = lease_based.try_into().unwrap();
assert_eq!(SelectorType::LeaseBased, selector_type);
let loadbased = "LoadBased";
let selector_type = loadbased.try_into().unwrap();
let load_based = "load_based";
let selector_type = load_based.try_into().unwrap();
assert_eq!(SelectorType::LoadBased, selector_type);
let load_based = "LoadBased";
let selector_type = load_based.try_into().unwrap();
assert_eq!(SelectorType::LoadBased, selector_type);
let unknown = "unknown";