Skip to content

Commit

Permalink
small cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
samansmink committed May 14, 2024
1 parent fb17ec8 commit 4efd591
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 9 deletions.
8 changes: 1 addition & 7 deletions extension_config.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,4 @@ duckdb_extension_load(deltatable
)

# Any extra extensions that should be built
duckdb_extension_load(httpfs)
#duckdb_extension_load(aws
# LOAD_TESTS
# GIT_URL https://github.com/duckdb/duckdb_aws
# GIT_TAG f7b8729f1cce5ada5d4add70e1486de50763fb97
# APPLY_PATCHES
# )
duckdb_extension_load(httpfs)
4 changes: 2 additions & 2 deletions src/functions/deltatable_scan.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -345,7 +345,7 @@ void DeltaMultiFileReader::FinalizeBind(const MultiFileReaderOptions &file_optio
// Get the metadata for this file
D_ASSERT(global_state->file_list);
const auto &snapshot = dynamic_cast<const DeltaTableSnapshot&>(*global_state->file_list);
auto &file_metadata = snapshot.metadata[reader_data.file_list_idx];
auto &file_metadata = snapshot.metadata[reader_data.file_list_idx.GetIndex()];

if (!file_metadata.partition_map.empty()) {
for (idx_t i = 0; i < global_column_ids.size(); i++) {
Expand Down Expand Up @@ -511,7 +511,7 @@ void DeltaMultiFileReader::FinalizeChunk(ClientContext &context, const MultiFile

// Get the metadata for this file
const auto &snapshot = dynamic_cast<const DeltaTableSnapshot&>(*global_state->file_list);
auto &metadata = snapshot.metadata[reader_data.file_list_idx];
auto &metadata = snapshot.metadata[reader_data.file_list_idx.GetIndex()];

if (metadata.selection_vector.get() && chunk.size() != 0) {
D_ASSERT(delta_global_state.file_row_number_idx != DConstants::INVALID_INDEX);
Expand Down
1 change: 1 addition & 0 deletions vcpkg.json
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
{
"dependencies": [
"openssl"
]
}

0 comments on commit 4efd591

Please sign in to comment.