apple/foundationdb
1
0
Fork 0
mirror of https://github.com/apple/foundationdb.git synced 2025-05-14 18:02:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

Address comments.

Browse Source
This commit is contained in:
Renxuan Wang 2022-01-20 18:09:01 -08:00
parent 02f72dd377
commit 8c49391c51
2 changed files with 9 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
fdbclient/CoordinationInterface.h
View File

@ -59,9 +59,9 @@ struct ClientLeaderRegInterface {
class ClusterConnectionString {
public:
ClusterConnectionString() {}
ClusterConnectionString(std::string const& connStr);
ClusterConnectionString(std::vector<NetworkAddress> coordinators, Key key);
ClusterConnectionString(std::vector<Hostname> hosts, Key key);
ClusterConnectionString(const std::string& connStr);
ClusterConnectionString(const std::vector<NetworkAddress>& coordinators, Key key);
ClusterConnectionString(const std::vector<Hostname>& hosts, Key key);
std::vector<NetworkAddress> const& coordinators() const { return coords; }
void addResolved(Hostname hostname, NetworkAddress address) {
@ -83,7 +83,7 @@ public:
private:
void parseConnString();
void parseKey(std::string const& key);
void parseKey(const std::string& key);
std::unordered_map<NetworkAddress, Hostname> networkAddressToHostname;
Key key, keyDesc;
std::string connectionString;

9
fdbclient/MonitorLeader.actor.cpp
View File

@ -172,7 +172,7 @@ void ClusterConnectionString::parseConnString() {
}
}
ClusterConnectionString::ClusterConnectionString(std::string const& connStr) {
ClusterConnectionString::ClusterConnectionString(const std::string& connStr) {
connectionString = trim(connStr);
parseConnString();
}
@ -399,7 +399,8 @@ TEST_CASE("/fdbclient/MonitorLeader/parseConnectionString/fuzz") {
return Void();
}
ClusterConnectionString::ClusterConnectionString(std::vector<NetworkAddress> servers, Key key) : coords(servers) {
ClusterConnectionString::ClusterConnectionString(const std::vector<NetworkAddress>& servers, Key key)
: coords(servers) {
std::string keyString = key.toString();
parseKey(keyString);
connectionString = keyString + "@";
@ -411,7 +412,7 @@ ClusterConnectionString::ClusterConnectionString(std::vector<NetworkAddress> ser
}
}
ClusterConnectionString::ClusterConnectionString(std::vector<Hostname> hosts, Key key)
ClusterConnectionString::ClusterConnectionString(const std::vector<Hostname>& hosts, Key key)
: hasUnresolvedHostnames(true), hostnames(hosts) {
std::string keyString = key.toString();
parseKey(keyString);
@ -424,7 +425,7 @@ ClusterConnectionString::ClusterConnectionString(std::vector<Hostname> hosts, Ke
}
}
void ClusterConnectionString::parseKey(std::string const& key) {
void ClusterConnectionString::parseKey(const std::string& key) {
// Check the structure of the given key, and fill in this->key and this->keyDesc
// The key must contain one (and only one) : character