Skip to content

Commit

Permalink
Use relative URIs for fragment consolidation.
Browse files Browse the repository at this point in the history
  • Loading branch information
shaunrd0 committed Jul 1, 2024
1 parent 2eddec8 commit 6957660
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions tiledb/sm/consolidator/fragment_consolidator.cc
Original file line number Diff line number Diff line change
Expand Up @@ -362,7 +362,7 @@ Status FragmentConsolidator::consolidate_fragments(
NDRange union_non_empty_domains;
std::unordered_set<std::string> to_consolidate_set;
for (auto& uri : fragment_uris) {
to_consolidate_set.emplace(uri);
to_consolidate_set.emplace(URI(uri).last_path_part());
}

// Make sure all fragments to consolidate are present
Expand All @@ -384,7 +384,8 @@ Status FragmentConsolidator::consolidate_fragments(

if (count != fragment_uris.size()) {
throw FragmentConsolidatorException(
"Cannot consolidate; Not all fragments could be found");
"Cannot consolidate; Found " + std::to_string(count) + " of " +
std::to_string(fragment_uris.size()) + " required fragments.");
}

FragmentConsolidationWorkspace cw(consolidator_memory_tracker_);
Expand Down

0 comments on commit 6957660

Please # to comment.