Skip to content

Commit

Permalink
Added a barrier to the DHT performance test for reliability.
Browse files Browse the repository at this point in the history
Improved buffered exchange performance slightly by ensuring more consistent flushing.
  • Loading branch information
Yucheng Low committed Sep 2, 2013
1 parent c583163 commit 8520d95
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion demoapps/rpc/dht_performance_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ int main(int argc, char ** argv) {
std::cout << "I am machine id " << dc.procid()
<< " in " << dc.numprocs() << " machines"<<std::endl;
dht<std::string, std::string> testdht(dc);

dc.barrier();
std::vector<std::pair<std::string, std::string> > data;
const size_t NUMSTRINGS = 10000;
const size_t strlen[4] = {16, 128, 1024, 10240};
Expand Down
2 changes: 2 additions & 0 deletions src/graphlab/rpc/buffered_exchange.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -136,6 +136,7 @@ namespace graphlab {
send_locks[index].unlock();
// complete the send
rpc.split_call_end(proc, prevarc);
rpc.dc().flush_soon(proc);
}
}
}
Expand All @@ -152,6 +153,7 @@ namespace graphlab {
}
send_locks[i].unlock();
}
rpc.dc().flush_soon();
rpc.full_barrier();
} // end of flush

Expand Down

0 comments on commit 8520d95

Please sign in to comment.