diff --git a/locator/network_topology_strategy.cc b/locator/network_topology_strategy.cc index b2ec91b366..03c55e4308 100644 --- a/locator/network_topology_strategy.cc +++ b/locator/network_topology_strategy.cc @@ -183,11 +183,11 @@ network_topology_strategy::calculate_natural_endpoints( void network_topology_strategy::validate_options() const { for (auto& c : _config_options) { - if (c.first == sstring("replication_factor")) + if (c.first == sstring("replication_factor")) { throw exceptions::configuration_exception( "replication_factor is an option for simple_strategy, not " "network_topology_strategy"); - + } validate_replication_factor(c.second); } } diff --git a/locator/simple_strategy.cc b/locator/simple_strategy.cc index 5e37c8e3c4..e4e2fd3ba2 100644 --- a/locator/simple_strategy.cc +++ b/locator/simple_strategy.cc @@ -54,8 +54,9 @@ size_t simple_strategy::get_replication_factor() const { void simple_strategy::validate_options() const { auto it = _config_options.find("replication_factor"); - if (it == _config_options.end()) + if (it == _config_options.end()) { throw exceptions::configuration_exception("SimpleStrategy requires a replication_factor strategy option."); + } validate_replication_factor(it->second); }