Merge pull request #24434 from nvartolomei/nv/CORE-7687

archival: calculate time based retention from first addressable segment
redpanda-data · Dec 4, 2024 · 61b306d · 61b306d
2 parents cddb5ef + a7d42d2
commit 61b306d
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/v/cluster/archival/retention_calculator.cc b/src/v/cluster/archival/retention_calculator.cc
@@ -109,7 +109,8 @@ std::optional<retention_calculator> retention_calculator::factory(
 
         if (
           manifest.size() > 0
-          && manifest.begin()->max_timestamp < oldest_allowed_timestamp) {
+          && manifest.first_addressable_segment()->max_timestamp
+               < oldest_allowed_timestamp) {
             strats.push_back(
               std::make_unique<time_based_strategy>(oldest_allowed_timestamp));
         }