From 695766009c18ace7a84725595c12fa89c368344b Mon Sep 17 00:00:00 2001 From: Shaun Reed Date: Fri, 21 Jun 2024 09:57:36 -0400 Subject: [PATCH] Use relative URIs for fragment consolidation. --- tiledb/sm/consolidator/fragment_consolidator.cc | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/tiledb/sm/consolidator/fragment_consolidator.cc b/tiledb/sm/consolidator/fragment_consolidator.cc index 2b8941f2a1dc..96f6b43cbc86 100644 --- a/tiledb/sm/consolidator/fragment_consolidator.cc +++ b/tiledb/sm/consolidator/fragment_consolidator.cc @@ -362,7 +362,7 @@ Status FragmentConsolidator::consolidate_fragments( NDRange union_non_empty_domains; std::unordered_set 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 @@ -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_);