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

Correct behavior of batch limit (was size_t)

Browse Source
This commit is contained in:
jeremy 2022-10-24 16:23:42 +02:00
parent 51e6802aa7
commit 8b9e7e2c65
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/storage/v3/shard_rsm.cpp
View File

@ -950,14 +950,10 @@ msgs::ReadResponses ShardRsm::HandleRead(msgs::ExpandOneRequest &&req) {
}
results.emplace_back(maybe_result.value());
if (batch_limit) { // #NoCommit can be done differently
--*batch_limit;
if (batch_limit < 0) {
if (batch_limit.has_value() && results.size() >= batch_limit.value()) {
break;
}
}
}
} else {
for (auto &src_vertex : req.src_vertices) {
// Get Vertex acc
@ -985,14 +981,11 @@ msgs::ReadResponses ShardRsm::HandleRead(msgs::ExpandOneRequest &&req) {
}
results.emplace_back(result.value());
if (batch_limit) { // #NoCommit can ebd one differently
--*batch_limit;
if (batch_limit < 0) {
if (batch_limit.has_value() && results.size() >= batch_limit.value()) {
break;
}
}
}
}
msgs::ExpandOneResponse resp{};
resp.success = action_successful;