bahman/scylla
1
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

locator: Fix minor formatting issues

Browse Source 
Fix minor formatting issues introduced in commit
b8211c436b
("locator/abstract_replication_strategy: Add validate_options() helper").

Signed-off-by: Pekka Enberg <penberg@cloudius-systems.com>
This commit is contained in:
Pekka Enberg
2015-09-01 08:59:29 +03:00
committed by Avi Kivity
parent 09da053bee
commit 47b84d9660
2 changed files with 4 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
locator/network_topology_strategy.cc
View File

@@ -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);
}
}

3
locator/simple_strategy.cc
View File

@@ -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);
}