From fbf028a4388f673dd84aa9e055ce1fbf2faa3823 Mon Sep 17 00:00:00 2001 From: Teon Banek Date: Mon, 22 May 2017 09:26:20 +0200 Subject: [PATCH] Update the benchmark to commit t1 before starting t2 Summary: The previous behavior was incorrect, since the second transaction cannot see the record created in the first, because it commited later. Reviewers: florijan, buda, dgleich Reviewed By: dgleich Subscribers: pullbot, matej.gradicek Differential Revision: https://phabricator.memgraph.io/D390 --- tests/benchmark/mvcc.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/benchmark/mvcc.cpp b/tests/benchmark/mvcc.cpp index 2c48a2a68..b8443b21a 100644 --- a/tests/benchmark/mvcc.cpp +++ b/tests/benchmark/mvcc.cpp @@ -18,8 +18,8 @@ void MvccMix(benchmark::State &state) { auto t1 = engine.begin(); mvcc::VersionList version_list(*t1); - auto t2 = engine.begin(); t1->commit(); + auto t2 = engine.begin(); state.ResumeTiming(); version_list.update(*t2);