tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Use 10us sleep while waiting on RPC

Browse Source 
Summary:
Usually a single RPC call takes 50us, which is a lot lower than our
millisecond sleep. Calling sleep with less than a ms duration may not
be really supported (due to timer granularity), but this change should
cause some sub-ms sleep.

Reviewers: florijan, msantl, mferencevic

Reviewed By: florijan

Subscribers: pullbot

Differential Revision: https://phabricator.memgraph.io/D1258
This commit is contained in:
Teon Banek 2018-02-28 13:59:12 +01:00
parent 5839dbc401
commit f963c54f77
1 changed files with 5 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/query/plan/operator.cpp
View File

@ -27,8 +27,8 @@
#include "utils/algorithm.hpp" #include "utils/algorithm.hpp"
#include "utils/exceptions.hpp" #include "utils/exceptions.hpp"
DEFINE_HIDDEN_int32(remote_pull_sleep, 1, DEFINE_HIDDEN_int32(remote_pull_sleep_micros, 10,
"Sleep between remote result pulling in milliseconds"); "Sleep between remote result pulling in microseconds");
// macro for the default implementation of LogicalOperator::Accept // macro for the default implementation of LogicalOperator::Accept
// that accepts the visitor and visits it's input_ operator // that accepts the visitor and visits it's input_ operator
@ -647,7 +647,7 @@ bool Expand::ExpandCursor::Pull(Frame &frame, Context &context) {
// We are still waiting for future edges, so sleep and fallthrough to // We are still waiting for future edges, so sleep and fallthrough to
// continue the loop. // continue the loop.
std::this_thread::sleep_for( std::this_thread::sleep_for(
std::chrono::milliseconds(FLAGS_remote_pull_sleep)); std::chrono::microseconds(FLAGS_remote_pull_sleep_micros));
} }
// we have re-initialized the edges, continue with the loop // we have re-initialized the edges, continue with the loop
@ -3181,7 +3181,7 @@ class PullRemoteCursor : public Cursor {
// If there aren't any local/remote results available, sleep. // If there aren't any local/remote results available, sleep.
std::this_thread::sleep_for( std::this_thread::sleep_for(
std::chrono::milliseconds(FLAGS_remote_pull_sleep)); std::chrono::microseconds(FLAGS_remote_pull_sleep_micros));
} }
} }
@ -3490,7 +3490,7 @@ class PullRemoteOrderByCursor : public Cursor {
if (!has_all_result) { if (!has_all_result) {
// If we don't have results from all workers, sleep before continuing. // If we don't have results from all workers, sleep before continuing.
std::this_thread::sleep_for( std::this_thread::sleep_for(
std::chrono::milliseconds(FLAGS_remote_pull_sleep)); std::chrono::microseconds(FLAGS_remote_pull_sleep_micros));
continue; continue;
} }