0b2d0b0d61
Summary: I think the rethrow as it was done in `.../clientes/common.hpp` is not correct as it could have caught any exception type descending from `BasicException`, copied it into `optional<BasicException>` and thus losing exact type and members of the original exception. I think that the proposed formulation of a rethrow in the `catch` does it properly. I made a small test for this behavior, see P11. Reviewers: teon.banek, mferencevic, mislav.bradac Reviewed By: teon.banek Subscribers: pullbot Differential Revision: https://phabricator.memgraph.io/D1298
86 lines
2.6 KiB
C++
86 lines
2.6 KiB
C++
#pragma once
|
|
|
|
#include <chrono>
|
|
#include <experimental/optional>
|
|
#include <map>
|
|
#include <random>
|
|
#include <string>
|
|
|
|
#include "communication/bolt/client.hpp"
|
|
#include "communication/bolt/v1/decoder/decoded_value.hpp"
|
|
#include "utils/exceptions.hpp"
|
|
#include "utils/timer.hpp"
|
|
|
|
namespace {
|
|
|
|
void PrintJsonDecodedValue(std::ostream &os,
|
|
const communication::bolt::DecodedValue &value) {
|
|
using communication::bolt::DecodedValue;
|
|
switch (value.type()) {
|
|
case DecodedValue::Type::Null:
|
|
os << "null";
|
|
break;
|
|
case DecodedValue::Type::Bool:
|
|
os << (value.ValueBool() ? "true" : "false");
|
|
break;
|
|
case DecodedValue::Type::Int:
|
|
os << value.ValueInt();
|
|
break;
|
|
case DecodedValue::Type::Double:
|
|
os << value.ValueDouble();
|
|
break;
|
|
case DecodedValue::Type::String:
|
|
os << "\"" << value.ValueString() << "\"";
|
|
break;
|
|
case DecodedValue::Type::List:
|
|
os << "[";
|
|
utils::PrintIterable(os, value.ValueList(), ", ",
|
|
[](auto &stream, const auto &item) {
|
|
PrintJsonDecodedValue(stream, item);
|
|
});
|
|
os << "]";
|
|
break;
|
|
case DecodedValue::Type::Map:
|
|
os << "{";
|
|
utils::PrintIterable(os, value.ValueMap(), ", ",
|
|
[](auto &stream, const auto &pair) {
|
|
PrintJsonDecodedValue(stream, {pair.first});
|
|
stream << ": ";
|
|
PrintJsonDecodedValue(stream, pair.second);
|
|
});
|
|
os << "}";
|
|
break;
|
|
default:
|
|
std::terminate();
|
|
}
|
|
}
|
|
|
|
template <typename TClient>
|
|
std::pair<communication::bolt::QueryData, int> ExecuteNTimesTillSuccess(
|
|
TClient &client, const std::string &query,
|
|
const std::map<std::string, communication::bolt::DecodedValue> ¶ms,
|
|
int max_attempts) {
|
|
static thread_local std::mt19937 pseudo_rand_gen_{std::random_device{}()};
|
|
static thread_local std::uniform_int_distribution<> rand_dist_{10, 50};
|
|
int failed_attempts{0};
|
|
while (true) {
|
|
try {
|
|
auto ret = client.Execute(query, params);
|
|
return {ret, failed_attempts};
|
|
} catch (const utils::BasicException &e) {
|
|
VLOG(0) << "Error: " << e.what();
|
|
if (++failed_attempts == max_attempts) {
|
|
LOG(WARNING) << query << " failed " << failed_attempts << "times";
|
|
throw;
|
|
}
|
|
utils::Timer t;
|
|
std::chrono::microseconds to_sleep(rand_dist_(pseudo_rand_gen_));
|
|
while (t.Elapsed() < to_sleep) {
|
|
cpu_relax();
|
|
}
|
|
}
|
|
}
|
|
}
|
|
|
|
} // namespace
|