mirror of
https://github.com/XRPLF/rippled.git
synced 2026-03-06 21:02:27 +00:00
3a8a18c2ca
This change replaces `void const*` by `uint256 const&` for database fetches. Object hashes are expressed using the `uint256` data type, and are converted to `void *` when calling the `fetch` or `fetchBatch` functions. However, in these fetch functions they are converted back to `uint256`, making the conversion process unnecessary. In a few cases the underlying pointer is needed, but that can then be easy obtained via `[hash variable].data()`.
96 lines
2.6 KiB
C++
96 lines
2.6 KiB
C++
#include <xrpl/nodestore/detail/DatabaseNodeImp.h>
|
|
|
|
namespace xrpl {
|
|
namespace NodeStore {
|
|
|
|
void
|
|
DatabaseNodeImp::store(NodeObjectType type, Blob&& data, uint256 const& hash, std::uint32_t)
|
|
{
|
|
storeStats(1, data.size());
|
|
|
|
auto obj = NodeObject::createObject(type, std::move(data), hash);
|
|
backend_->store(obj);
|
|
}
|
|
|
|
void
|
|
DatabaseNodeImp::asyncFetch(
|
|
uint256 const& hash,
|
|
std::uint32_t ledgerSeq,
|
|
std::function<void(std::shared_ptr<NodeObject> const&)>&& callback)
|
|
{
|
|
Database::asyncFetch(hash, ledgerSeq, std::move(callback));
|
|
}
|
|
|
|
std::shared_ptr<NodeObject>
|
|
DatabaseNodeImp::fetchNodeObject(
|
|
uint256 const& hash,
|
|
std::uint32_t,
|
|
FetchReport& fetchReport,
|
|
bool duplicate)
|
|
{
|
|
std::shared_ptr<NodeObject> nodeObject = nullptr;
|
|
Status status;
|
|
|
|
try
|
|
{
|
|
status = backend_->fetch(hash, &nodeObject);
|
|
}
|
|
catch (std::exception const& e)
|
|
{
|
|
JLOG(j_.fatal()) << "fetchNodeObject " << hash
|
|
<< ": Exception fetching from backend: " << e.what();
|
|
Rethrow();
|
|
}
|
|
|
|
switch (status)
|
|
{
|
|
case ok:
|
|
case notFound:
|
|
break;
|
|
case dataCorrupt:
|
|
JLOG(j_.fatal()) << "fetchNodeObject " << hash << ": nodestore data is corrupted";
|
|
break;
|
|
default:
|
|
JLOG(j_.warn()) << "fetchNodeObject " << hash << ": backend returns unknown result "
|
|
<< status;
|
|
break;
|
|
}
|
|
|
|
if (nodeObject)
|
|
fetchReport.wasFound = true;
|
|
|
|
return nodeObject;
|
|
}
|
|
|
|
std::vector<std::shared_ptr<NodeObject>>
|
|
DatabaseNodeImp::fetchBatch(std::vector<uint256> const& hashes)
|
|
{
|
|
using namespace std::chrono;
|
|
auto const before = steady_clock::now();
|
|
|
|
// Get the node objects that match the hashes from the backend. To protect
|
|
// against the backends returning fewer or more results than expected, the
|
|
// container is resized to the number of hashes.
|
|
auto results = backend_->fetchBatch(hashes).first;
|
|
XRPL_ASSERT(
|
|
results.size() == hashes.size() || results.empty(),
|
|
"number of output objects either matches number of input hashes or is empty");
|
|
results.resize(hashes.size());
|
|
for (size_t i = 0; i < results.size(); ++i)
|
|
{
|
|
if (!results[i])
|
|
{
|
|
JLOG(j_.error()) << "fetchBatch - "
|
|
<< "record not found in db. hash = " << strHex(hashes[i]);
|
|
}
|
|
}
|
|
|
|
auto fetchDurationUs =
|
|
std::chrono::duration_cast<std::chrono::microseconds>(steady_clock::now() - before).count();
|
|
updateFetchMetrics(hashes.size(), 0, fetchDurationUs);
|
|
return results;
|
|
}
|
|
|
|
} // namespace NodeStore
|
|
} // namespace xrpl
|