Apply clang-tidy feedback

This commit is contained in:
Tyler Neely 2022-12-05 13:43:20 +00:00
parent 1b458ebc41
commit ca3f748325
2 changed files with 5 additions and 3 deletions

View File

@ -14,6 +14,7 @@
#include <condition_variable>
#include <functional>
#include <mutex>
#include <optional>
#include <vector>
namespace memgraph::io {

View File

@ -359,7 +359,8 @@ class RequestRouter : public RequestRouterInterface {
std::vector<msgs::GetPropertiesResultRow> GetProperties(msgs::GetPropertiesRequest requests) override {
// create requests
std::vector<ShardRequestState<msgs::GetPropertiesRequest>> unsent_requests = RequestsForGetProperties(requests);
std::vector<ShardRequestState<msgs::GetPropertiesRequest>> unsent_requests =
RequestsForGetProperties(std::move(requests));
// begin all requests in parallel
RunningRequests<msgs::GetPropertiesRequest> running_requests = {};
@ -530,14 +531,14 @@ class RequestRouter : public RequestRouterInterface {
}
std::vector<ShardRequestState<msgs::GetPropertiesRequest>> RequestsForGetProperties(
const msgs::GetPropertiesRequest &request) {
msgs::GetPropertiesRequest &&request) {
std::map<Shard, msgs::GetPropertiesRequest> per_shard_request_table;
auto top_level_rqst_template = request;
top_level_rqst_template.transaction_id = transaction_id_;
top_level_rqst_template.vertex_ids.clear();
top_level_rqst_template.vertices_and_edges.clear();
for (auto &vertex : request.vertex_ids) {
for (auto &&vertex : request.vertex_ids) {
auto shard =
shards_map_.GetShardForKey(vertex.first.id, storage::conversions::ConvertPropertyVector(vertex.second));
if (!per_shard_request_table.contains(shard)) {