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

locator: Use local members in ..._replication_strategy constructors

Browse Source 
The `config_options` arg had been used to initialize `_config_options`
field of the base abstract_replication_strategy class, so it's more
idiomatic to use the latter. Also it makes next patches simpler.

Signed-off-by: Pavel Emelyanov <xemul@scylladb.com>
This commit is contained in:
Pavel Emelyanov
2023-12-21 12:55:22 +03:00
parent 060b16f987
commit ecbafd81f2
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
locator/network_topology_strategy.cc
View File

@@ -33,7 +33,7 @@ network_topology_strategy::network_topology_strategy(
const replication_strategy_config_options& config_options) :
abstract_replication_strategy(config_options,
replication_strategy_type::network_topology) {
auto opts = config_options;
auto opts = _config_options;
process_tablet_options(*this, opts);
for (auto& config_pair : opts) {

2
locator/simple_strategy.cc
View File

@@ -20,7 +20,7 @@ namespace locator {
simple_strategy::simple_strategy(const replication_strategy_config_options& config_options) :
abstract_replication_strategy(config_options, replication_strategy_type::simple) {
for (auto& config_pair : config_options) {
for (auto& config_pair : _config_options) {
auto& key = config_pair.first;
auto& val = config_pair.second;