Skip to content

Commit

Permalink
minor fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
samansmink committed Oct 25, 2024
1 parent 1261ea9 commit cc4eea6
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion benchmark/benchmark.Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ bench-run-tpch-sf1-parquet: bench-output-dir
bench-run-tpch-sf1-duckdb: bench-output-dir
./build/release/benchmark/benchmark_runner --root-dir './' 'benchmark/tpch/sf1/local/duckdb/$(BENCHMARK_PATTERN)' 2>&1 | tee benchmark_results/tpch-sf1-duckdb.csv
# COMPARES TPCH SF1 on parquet file vs on delta files vs on duckdb files
bench-run-tpch-sf1: bench-run-tpch-sf1-delta bench-run-tpch-sf1-parquet bench-run-tpch-sf1-attach
bench-run-tpch-sf1: bench-run-tpch-sf1-delta bench-run-tpch-sf1-parquet bench-run-tpch-sf1-delta-attach

###
# TPCDS
Expand Down
3 changes: 1 addition & 2 deletions src/storage/delta_catalog.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -71,8 +71,7 @@ optional_idx DeltaCatalog::GetCatalogVersion(ClientContext &context) {
return delta_transaction.table_entry->snapshot->version;
}

// FIXME: this is not allowed
return optional_idx::Invalid();
return {};
}

DatabaseSize DeltaCatalog::GetDatabaseSize(ClientContext &context) {
Expand Down

0 comments on commit cc4eea6

Please sign in to comment.