dn724/dncurrency
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Do not complain when a query timeout, if replies were received

Browse source
This commit is contained in:
Dimitrios Siganos 2024-03-07 15:14:52 +00:00
commit 4f2f66bbe2
3 changed files with 15 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

1
nano/lib/stats_enums.hpp
View file

@ -337,6 +337,7 @@ enum class detail : uint8_t
query_duplicate,
rep_timeout,
query_timeout,
query_completion,
crawl_aggressive,
crawl_normal,

16
nano/node/repcrawler.cpp
View file

@ -211,8 +211,16 @@ void nano::rep_crawler::cleanup ()
erase_if (queries, [this] (query_entry const & query) {
if (nano::elapsed (query.time, config.query_timeout))
{
logger.debug (nano::log::type::rep_crawler, "Aborting unresponsive query for block {} from {}", query.hash.to_string (), query.channel->to_string ());
stats.inc (nano::stat::type::rep_crawler, nano::stat::detail::query_timeout);
if (query.replies == 0)
{
logger.debug (nano::log::type::rep_crawler, "Aborting unresponsive query for block {} from {}", query.hash.to_string (), query.channel->to_string ());
stats.inc (nano::stat::type::rep_crawler, nano::stat::detail::query_timeout);
}
else
{
logger.debug (nano::log::type::rep_crawler, "Completion of query with {} replies for block {} from {}", query.replies, query.hash.to_string (), query.channel->to_string ());
stats.inc (nano::stat::type::rep_crawler, nano::stat::detail::query_completion);
}
return true; // Erase
}
return false;
@ -401,7 +409,9 @@ bool nano::rep_crawler::process (std::shared_ptr<nano::vote> const & vote, std::
// TODO: Track query response time
responses.push_back ({ channel, vote });
queries.modify (it, [] (query_entry & e) {
e.replies++;
});
condition.notify_all ();
return true; // Found and processed
}

1
nano/node/repcrawler.hpp
View file

@ -141,6 +141,7 @@ private:
nano::block_hash hash;
std::shared_ptr<nano::transport::channel> channel;
std::chrono::steady_clock::time_point time{ std::chrono::steady_clock::now () };
unsigned int replies{ 0 }; // number of replies to the query
};
// clang-format off