Skip to content
This repository was archived by the owner on May 9, 2024. It is now read-only.

Fix taxi_bench when USE_HOT_DATA is enabled #587

Merged
merged 1 commit into from
Jul 19, 2023
Merged
Changes from all commits
Commits
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
4 changes: 2 additions & 2 deletions omniscidb/Benchmarks/taxi/taxi_full_bench.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -22,9 +22,7 @@ using namespace TestHelpers::ArrowSQLRunner;

// when we want to measure storage latencies, read the csv files before starting the
// benchmark
#ifndef USE_HOT_DATA
std::vector<std::shared_ptr<arrow::Table>> g_taxi_data_files;
#endif

std::istream& operator>>(std::istream& in, ExecutorDeviceType& device_type) {
std::string token;
Expand Down Expand Up @@ -162,6 +160,7 @@ static void createTaxiTable() {
}
}

#ifndef USE_HOT_DATA
static std::vector<std::shared_ptr<arrow::Table>> readTaxiFilesCsv(
const ColumnInfoList& col_infos) {
std::vector<std::shared_ptr<arrow::Table>> taxi_arrow_data;
Expand Down Expand Up @@ -198,6 +197,7 @@ static std::vector<std::shared_ptr<arrow::Table>> readTaxiFilesCsv(
std::cout << "Read taxi csv files in " << time << " ms" << std::endl;
return taxi_arrow_data;
}
#endif

static void loadTaxiArrowData() {
for (auto& data_file : g_taxi_data_files) {
Expand Down