diff options
| author | osidorkin <[email protected]> | 2025-04-11 10:27:37 +0300 |
|---|---|---|
| committer | osidorkin <[email protected]> | 2025-04-11 10:41:10 +0300 |
| commit | 60f4418667ea2f14124a22f7e4d9d5bcf5eabce4 (patch) | |
| tree | 4fa0e31e46373f9cd17c54cc7c22cc0041060290 | |
| parent | 49c1928dd8000db78dee82ab203d4259400a73d1 (diff) | |
YT-24781: min_sync_queue_replica_count should be greater than 0. Fix typo in error message
commit_hash:44a685c3d08923fdee491447c37e323fda983471
| -rw-r--r-- | yt/yt/client/tablet_client/config.cpp | 13 |
1 files changed, 6 insertions, 7 deletions
diff --git a/yt/yt/client/tablet_client/config.cpp b/yt/yt/client/tablet_client/config.cpp index dc6a909d223..83dde143e04 100644 --- a/yt/yt/client/tablet_client/config.cpp +++ b/yt/yt/client/tablet_client/config.cpp @@ -79,14 +79,18 @@ void TReplicatedTableOptions::Register(TRegistrar registrar) { registrar.Parameter("max_sync_replica_count", &TThis::MaxSyncReplicaCount) .Alias("sync_replica_count") - .Optional(); + .Optional() + .GreaterThanOrEqual(0); registrar.Parameter("min_sync_replica_count", &TThis::MinSyncReplicaCount) - .Optional(); + .Optional() + .GreaterThanOrEqual(0); registrar.Parameter("max_sync_queue_replica_count", &TThis::MaxSyncQueueReplicaCount) .Optional() + .GreaterThanOrEqual(2) .DontSerializeDefault(); registrar.Parameter("min_sync_queue_replica_count", &TThis::MinSyncQueueReplicaCount) .Optional() + .GreaterThanOrEqual(1) .DontSerializeDefault(); registrar.Parameter("enable_replicated_table_tracker", &TThis::EnableReplicatedTableTracker) @@ -113,11 +117,6 @@ void TReplicatedTableOptions::Register(TRegistrar registrar) THROW_ERROR_EXCEPTION("\"min_sync_replica_count\" must be less or equal to \"max_sync_replica_count\""); } - if (config->MaxSyncQueueReplicaCount && config->MaxSyncQueueReplicaCount < 2) { - THROW_ERROR_EXCEPTION("\"max_sync_queue_replica_count\" canot be less than 2, actual: %v", - config->MaxSyncQueueReplicaCount); - } - if (config->MaxSyncQueueReplicaCount && config->MinSyncQueueReplicaCount && config->MinSyncQueueReplicaCount > config->MaxSyncQueueReplicaCount) |