Skip to content
This repository has been archived by the owner on Feb 20, 2023. It is now read-only.

Synchronous Replication #1472

Merged
merged 103 commits into from
Mar 9, 2021
Merged
Show file tree
Hide file tree
Changes from 99 commits
Commits
Show all changes
103 commits
Select commit Hold shift + click to select a range
641bf6c
Checkpoint. New replication manager.
lmwnshn Nov 2, 2020
c27168f
tmp unpush
lmwnshn Nov 2, 2020
3854236
Checkpoint. Fix messenger_test.
lmwnshn Nov 4, 2020
b47569b
Refactor the Messenger with Matt's suggestion.
lmwnshn Nov 4, 2020
6e13413
Checkpoint.
lmwnshn Nov 5, 2020
0de754e
Checkpoint. Replication locally via noisepage. Pretty sure tests brok…
lmwnshn Nov 5, 2020
afb1280
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Dec 28, 2020
5584bd4
Switch to sanctionedsharedptr for logger.
lmwnshn Dec 28, 2020
ea976b4
Nuke dead code.
lmwnshn Dec 28, 2020
26c2e87
Eliminate potential race in messenger_test.
lmwnshn Dec 29, 2020
80f7767
Checkpoint.
lmwnshn Dec 29, 2020
375fb14
Checkpoint.
lmwnshn Dec 29, 2020
0d4a470
Checkpoint.
lmwnshn Dec 29, 2020
410cd96
Checkpoint.
lmwnshn Dec 31, 2020
73cf888
Add Tianlei's ReplicationLogProvider port.
tpan496 Jan 4, 2021
ef64dd5
Add support for simple replication.
lmwnshn Jan 4, 2021
bd407cb
Checkpoint. Slight progress. ty matt
lmwnshn Jan 14, 2021
5ca79ba
Fix the replica not returning serialized buffers to the empty buffer …
lmwnshn Feb 4, 2021
4d83efd
unpush me later.
lmwnshn Feb 9, 2021
380abb4
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 11, 2021
ed0c6bd
Checkpoint.
lmwnshn Feb 11, 2021
e9803b6
Checkpoint. Add dummy replication_get_last_record_id builtin.
lmwnshn Feb 14, 2021
0acb22d
Checkpoint. Nuke the ITP legacy stuff. Hook up replication manager an…
lmwnshn Feb 14, 2021
4739b65
Checkpoint. Shutdown, except we had to add some DBMain stuff that I s…
lmwnshn Feb 15, 2021
7d7d2ba
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 15, 2021
730f62c
Checkpoint. recovery_test is failing?
lmwnshn Feb 15, 2021
3bf0b14
Minor cleanup.
lmwnshn Feb 15, 2021
f17cbe8
Simplify cleanup logic.
lmwnshn Feb 15, 2021
e6a738f
Fix RecoveryManager race-y logic. Add documentation and warnings.
lmwnshn Feb 16, 2021
a458409
Merge branch 'master' into replication
lmwnshn Feb 16, 2021
dde1ef6
Documentation.
lmwnshn Feb 17, 2021
6c61675
add retention policy
tpan496 Feb 16, 2021
72fcf94
Checkpoint. Doxygen fixes and reverting to changes that I'm not comfo…
lmwnshn Feb 17, 2021
95f2aa4
Merge branch 'master' into replication
lmwnshn Feb 17, 2021
9ec03e4
Format.
lmwnshn Feb 17, 2021
8741991
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 17, 2021
63fae63
Small fix after removing unused variable.
lmwnshn Feb 17, 2021
05b0d56
Clang-tidy doesn't like excessive padding.
lmwnshn Feb 17, 2021
3e04d56
Update db_main comments.
lmwnshn Feb 17, 2021
8336a14
Update settings comment.
lmwnshn Feb 17, 2021
0d26d03
Update RecoveryManager comment.
lmwnshn Feb 17, 2021
2521f90
Move TODO around.
lmwnshn Feb 17, 2021
b939e0e
Update comment.
lmwnshn Feb 17, 2021
85d8908
Add explicit.
lmwnshn Feb 17, 2021
03f468b
Fix parameter naming. I swear I did this already.
lmwnshn Feb 18, 2021
edfcff9
Fix transaction_manager weirdness. Still a bit iffy on this...
lmwnshn Feb 18, 2021
8d2c807
Checkpoint.
lmwnshn Feb 18, 2021
861cf5f
Checkpoint.
lmwnshn Feb 18, 2021
1b5af2a
Checkpoint.
lmwnshn Feb 18, 2021
f5ef916
gitignore...
lmwnshn Feb 18, 2021
99dc705
Checkpoint.
lmwnshn Feb 19, 2021
51e2ffa
Fix mini runners.
lmwnshn Feb 19, 2021
4d9ff46
Merge branch 'master' into replication
lmwnshn Feb 19, 2021
507df4c
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 19, 2021
0c86727
Merge branch 'replication' of github.com:lmwnshn/noisepage into repli…
lmwnshn Feb 20, 2021
23d3fda
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 20, 2021
5e014f1
Add ninja job pools. Num logical cores = compile pool + link pool (1)…
lmwnshn Feb 21, 2021
c371699
Merge branch 'ninja_link' into replication
lmwnshn Feb 21, 2021
554961e
Merge branch 'master' into replication
lmwnshn Feb 22, 2021
a2e48db
Store Messenger IPC locally instead of in /tmp. Also delete IPC file …
lmwnshn Feb 22, 2021
8cd8c57
Don't hang if the other node dies in sync replication.
lmwnshn Feb 22, 2021
0c515a6
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Feb 25, 2021
b568f14
Format.
lmwnshn Feb 25, 2021
9430157
Fix bplustree.h redefined constant shared with bwtree.h.
lmwnshn Feb 25, 2021
e040415
Merge branch 'bplustree_fix' into replication
lmwnshn Feb 25, 2021
3c81a4f
Lint.
lmwnshn Feb 25, 2021
3a1684f
Merge branch 'bplustree_fix' into replication
lmwnshn Feb 25, 2021
86b2a11
Add logging for enabling/disabling replication.
lmwnshn Feb 26, 2021
1042ae7
Fix bug in replica_sync.
lmwnshn Feb 26, 2021
47a804e
Add (what I hope is) the description of the right message in Replicat…
lmwnshn Feb 26, 2021
6fa6ec2
Remove dead code.
lmwnshn Mar 4, 2021
cf9e5b3
Refactor PrepareForSerialization.
lmwnshn Mar 4, 2021
2b56dee
Remove dead message.
lmwnshn Mar 4, 2021
612d3e5
Remove superfluous unlock.
lmwnshn Mar 4, 2021
107053f
Remove debug logging.
lmwnshn Mar 4, 2021
0e05526
Checkpoint. First pass at separating out primary/replica logic in Rep…
lmwnshn Mar 4, 2021
5e6b544
Checkpoint. Add documentation, including for the BufferedLogWriter mo…
lmwnshn Mar 4, 2021
6414138
Refactor json message keys into a ReplicateBufferMessage container cl…
lmwnshn Mar 4, 2021
5c90e42
Merge branch 'master' of github.com:cmu-db/noisepage into replication
lmwnshn Mar 4, 2021
d24cf2c
Add warning comment on message delivery failure.
lmwnshn Mar 4, 2021
146c903
Fix clang-tidy.
lmwnshn Mar 4, 2021
e89354e
Stash the "primary" constant into the PrimaryReplicationManager.
lmwnshn Mar 5, 2021
e3547bc
Move the replication test to the top.
lmwnshn Mar 5, 2021
caba0b1
Try switching localhost to 127.0.0.1.
lmwnshn Mar 5, 2021
acc2cda
Merge branch 'master' into replication
lmwnshn Mar 6, 2021
8142da4
Add top information to print_docker_info.
lmwnshn Mar 6, 2021
61d9aad
Try disabling ccache.
lmwnshn Mar 6, 2021
04ffc5d
Revert "Try disabling ccache."
lmwnshn Mar 6, 2021
7a13faf
Revert "Stash the "primary" constant into the PrimaryReplicationManag…
lmwnshn Mar 6, 2021
34685bc
Sin by sleeping. CI...
lmwnshn Mar 7, 2021
466deee
Try fixing a potential race between ListenForConnection() and the con…
lmwnshn Mar 7, 2021
851502a
blyat
lmwnshn Mar 7, 2021
4a9936e
Nope. Didn't work. Back to having a 30 second sleep.
lmwnshn Mar 8, 2021
e027b92
It might have been the SNDRCV timeout of 1 second. Try bumping that u…
lmwnshn Mar 8, 2021
f6627fa
I AM A MONKEY OF THE HIGHEST ORDER.
lmwnshn Mar 8, 2021
4d24ade
Avoid signalling cvars if there are no routers to add.
lmwnshn Mar 8, 2021
bd80a73
Merge branch 'master' into replication
mbutrovich Mar 8, 2021
649f051
Switch back to the model where the Messenger is responsible for invok…
lmwnshn Mar 8, 2021
4560cf5
Merge remote-tracking branch 'origin/replication' into replication
lmwnshn Mar 8, 2021
fcba2ca
Rearchitect the server loop structure to rely on key_message_type ins…
lmwnshn Mar 9, 2021
e85036f
Add some messenger documentation.
lmwnshn Mar 9, 2021
8898968
Add documentation on why routers are added with a level of indirection.
lmwnshn Mar 9, 2021
9868193
Delete dead comment.
lmwnshn Mar 9, 2021
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 13 additions & 1 deletion CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@
# HEADER Dependencies.
# HEADER noisepage libraries.
# HEADER noisepage binary.
# HEADER noisepage miscellaneous files needed for operation.
# HEADER util_static and util_shared libraries.
# HEADER hack_bytecode_handlers_ir target.
# HEADER gen_opt_bc binary.
Expand Down Expand Up @@ -697,6 +698,18 @@ target_compile_options(noisepage PRIVATE "-Werror" "-Wall")
target_link_libraries(noisepage noisepage_static)
set_target_properties(noisepage PROPERTIES CXX_EXTENSIONS OFF ENABLE_EXPORTS ON)

#######################################################################################################################
# HEADER noisepage miscellaneous files needed for operation.
# replication.config : The identities, hostnames, and ports, of all primaries and replicas.
#######################################################################################################################

# replication.config specifies the identities, hostnames, and ports, and is needed when replication is enabled.
add_custom_command(
TARGET noisepage
POST_BUILD
COMMAND ${CMAKE_COMMAND} -E copy "${BUILD_SUPPORT_DIR}/data/replication.config" "${CMAKE_BINARY_DIR}/bin/replication.config"
)

#######################################################################################################################
# HEADER util_static and util_shared libraries.
# util_static : table_generator and table_reader for tests and benchmarks.
Expand Down Expand Up @@ -781,7 +794,6 @@ add_dependencies(noisepage_static bitcode_generator)
#######################################################################################################################
# HEADER tpl binary.
# tpl : Standalone binary for testing .tpl files and commands.
# Also generates POST_BUILD bytecode_handlers_ir.bc.
#######################################################################################################################

add_executable(tpl util/execution/tpl.cpp)
Expand Down
4 changes: 4 additions & 0 deletions Jenkinsfile
Original file line number Diff line number Diff line change
Expand Up @@ -138,6 +138,7 @@ pipeline {
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15722', label: 'Kill PID(15722)'
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15723', label: 'Kill PID(15723)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
sh script: 'cd build/bin && PYTHONPATH=../.. timeout 20m python3 -m script.testing.replication.tests_simple --build-type=debug', label: 'Replication (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=debug --query-mode=simple', label: 'UnitTest (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=debug --query-mode=extended', label: 'UnitTest (Extended)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=debug --query-mode=extended -a "pipeline_metrics_enable=True" -a "pipeline_metrics_sample_rate=100" -a "counters_enable=True" -a "query_trace_metrics_enable=True"', label: 'UnitTest (Extended with pipeline metrics, counters, and query trace metrics)'
Expand Down Expand Up @@ -235,6 +236,7 @@ pipeline {
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15722', label: 'Kill PID(15722)'
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15723', label: 'Kill PID(15723)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
sh script: 'cd build/bin && PYTHONPATH=../.. timeout 20m python3 -m script.testing.replication.tests_simple --build-type=debug', label: 'Replication (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=debug --query-mode=simple', label: 'UnitTest (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=debug --query-mode=extended', label: 'UnitTest (Extended)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
Expand Down Expand Up @@ -273,6 +275,7 @@ pipeline {
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15722', label: 'Kill PID(15722)'
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15723', label: 'Kill PID(15723)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
sh script: 'cd build/bin && PYTHONPATH=../.. timeout 20m python3 -m script.testing.replication.tests_simple --build-type=release', label: 'Replication (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=release --query-mode=simple', label: 'UnitTest (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=release --query-mode=extended', label: 'UnitTest (Extended)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
Expand Down Expand Up @@ -315,6 +318,7 @@ pipeline {
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15722', label: 'Kill PID(15722)'
sh script: 'cd build && timeout 10s sudo python3 -B ../script/testing/kill_server.py 15723', label: 'Kill PID(15723)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
sh script: 'cd build/bin && PYTHONPATH=../.. timeout 20m python3 -m script.testing.replication.tests_simple --build-type=release', label: 'Replication (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=release --query-mode=simple', label: 'UnitTest (Simple)'
sh script: 'cd build && PYTHONPATH=.. timeout 20m python3 -m script.testing.junit --build-type=release --query-mode=extended', label: 'UnitTest (Extended)'
sh script: 'sudo lsof -i -P -n | grep LISTEN || true', label: 'Check ports.'
Expand Down
17 changes: 9 additions & 8 deletions benchmark/integration/tpcc_benchmark.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ class TPCCBenchmark : public benchmark::Fixture {
storage::RecordBufferSegmentPool buffer_pool_{buffersegment_size_limit_, buffersegment_reuse_limit_};
std::default_random_engine generator_;
storage::LogManager *log_manager_ = DISABLED; // logging enabled will override this value
common::ConcurrentBlockingQueue<storage::BufferedLogWriter *> empty_buffer_queue_;

// Settings for log manager
const uint64_t num_log_buffers_ = 100;
Expand Down Expand Up @@ -173,10 +174,10 @@ BENCHMARK_DEFINE_F(TPCCBenchmark, ScaleFactor4WithLogging)(benchmark::State &sta
unlink(noisepage::BenchmarkConfig::logfile_path.data());
thread_registry_ = new common::DedicatedThreadRegistry(DISABLED);
// we need transactions, TPCC database, and GC
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(thread_registry_));
log_manager_ = new storage::LogManager(
noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_, log_serialization_interval_,
log_persist_interval_, log_persist_threshold_, common::ManagedPointer(&buffer_pool_),
common::ManagedPointer(&empty_buffer_queue_), DISABLED, common::ManagedPointer(thread_registry_));
log_manager_->Start();
transaction::TimestampManager timestamp_manager;
transaction::DeferredActionManager deferred_action_manager{common::ManagedPointer(&timestamp_manager)};
Expand Down Expand Up @@ -275,10 +276,10 @@ BENCHMARK_DEFINE_F(TPCCBenchmark, ScaleFactor4WithLoggingAndMetrics)(benchmark::
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry{common::ManagedPointer(metrics_manager)};
// we need transactions, TPCC database, and GC
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(thread_registry_));
log_manager_ = new storage::LogManager(
noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_, log_serialization_interval_,
log_persist_interval_, log_persist_threshold_, common::ManagedPointer(&buffer_pool_),
common::ManagedPointer(&empty_buffer_queue_), DISABLED, common::ManagedPointer(thread_registry_));
log_manager_->Start();
transaction::TimestampManager timestamp_manager;
transaction::DeferredActionManager deferred_action_manager{common::ManagedPointer(&timestamp_manager)};
Expand Down
46 changes: 26 additions & 20 deletions benchmark/metrics/logging_metrics_benchmark.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,7 @@ class LoggingMetricsBenchmark : public benchmark::Fixture {
const std::chrono::microseconds gc_period_{1000};
const std::chrono::microseconds metrics_period_{10000};
common::DedicatedThreadRegistry *thread_registry_ = nullptr;
common::ConcurrentBlockingQueue<storage::BufferedLogWriter *> empty_buffer_queue_;

// Settings for log manager
const uint64_t num_log_buffers_ = 100;
Expand All @@ -55,10 +56,11 @@ BENCHMARK_DEFINE_F(LoggingMetricsBenchmark, TPCCish)(benchmark::State &state) {
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry(common::ManagedPointer(metrics_manager));

log_manager_ = new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_),
common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(&empty_buffer_queue_),
DISABLED, common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_->Start();
LargeDataTableBenchmarkObject tested(attr_sizes_, initial_table_size_, txn_length, insert_update_select_ratio,
&block_store_, &buffer_pool_, &generator_, true, log_manager_);
Expand Down Expand Up @@ -106,10 +108,11 @@ BENCHMARK_DEFINE_F(LoggingMetricsBenchmark, HighAbortRate)(benchmark::State &sta
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry(common::ManagedPointer(metrics_manager));

log_manager_ = new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_),
common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(&empty_buffer_queue_),
DISABLED, common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_->Start();
LargeDataTableBenchmarkObject tested(attr_sizes_, 1000, txn_length, insert_update_select_ratio, &block_store_,
&buffer_pool_, &generator_, true, log_manager_);
Expand Down Expand Up @@ -157,10 +160,11 @@ BENCHMARK_DEFINE_F(LoggingMetricsBenchmark, SingleStatementInsert)(benchmark::St
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry(common::ManagedPointer(metrics_manager));

log_manager_ = new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_),
common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(&empty_buffer_queue_),
DISABLED, common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_->Start();
LargeDataTableBenchmarkObject tested(attr_sizes_, 0, txn_length, insert_update_select_ratio, &block_store_,
&buffer_pool_, &generator_, true, log_manager_);
Expand Down Expand Up @@ -208,10 +212,11 @@ BENCHMARK_DEFINE_F(LoggingMetricsBenchmark, SingleStatementUpdate)(benchmark::St
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry(common::ManagedPointer(metrics_manager));

log_manager_ = new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_),
common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(&empty_buffer_queue_),
DISABLED, common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_->Start();
LargeDataTableBenchmarkObject tested(attr_sizes_, initial_table_size_, txn_length, insert_update_select_ratio,
&block_store_, &buffer_pool_, &generator_, true, log_manager_);
Expand Down Expand Up @@ -259,10 +264,11 @@ BENCHMARK_DEFINE_F(LoggingMetricsBenchmark, SingleStatementSelect)(benchmark::St
metrics_manager->EnableMetric(metrics::MetricsComponent::LOGGING);
thread_registry_ = new common::DedicatedThreadRegistry(common::ManagedPointer(metrics_manager));

log_manager_ = new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_),
common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_ =
new storage::LogManager(noisepage::BenchmarkConfig::logfile_path.data(), num_log_buffers_,
log_serialization_interval_, log_persist_interval_, log_persist_threshold_,
common::ManagedPointer(&buffer_pool_), common::ManagedPointer(&empty_buffer_queue_),
DISABLED, common::ManagedPointer<common::DedicatedThreadRegistry>(thread_registry_));
log_manager_->Start();
LargeDataTableBenchmarkObject tested(attr_sizes_, initial_table_size_, txn_length, insert_update_select_ratio,
&block_store_, &buffer_pool_, &generator_, true, log_manager_);
Expand Down
Loading