From a30fe4f03d1999832155c2d291e1c542f4f13731 Mon Sep 17 00:00:00 2001 From: Yukang-Lian Date: Wed, 4 Jun 2025 15:57:52 +0800 Subject: [PATCH] 1 --- be/src/cloud/cloud_cumulative_compaction.cpp | 1 - be/src/olap/cumulative_compaction.cpp | 1 - be/src/olap/tablet.cpp | 3 ++- 3 files changed, 2 insertions(+), 3 deletions(-) diff --git a/be/src/cloud/cloud_cumulative_compaction.cpp b/be/src/cloud/cloud_cumulative_compaction.cpp index fccea4a0925184..8759060eb136d6 100644 --- a/be/src/cloud/cloud_cumulative_compaction.cpp +++ b/be/src/cloud/cloud_cumulative_compaction.cpp @@ -53,7 +53,6 @@ Status CloudCumulativeCompaction::prepare_compact() { Defer defer_set_st([&] { if (!st.ok()) { cloud_tablet()->set_last_cumu_compaction_status(st.to_string()); - cloud_tablet()->set_last_cumu_compaction_failure_time(UnixMillis()); } }); if (_tablet->tablet_state() != TABLET_RUNNING && diff --git a/be/src/olap/cumulative_compaction.cpp b/be/src/olap/cumulative_compaction.cpp index 488937257bdc09..a0746997128e46 100644 --- a/be/src/olap/cumulative_compaction.cpp +++ b/be/src/olap/cumulative_compaction.cpp @@ -85,7 +85,6 @@ Status CumulativeCompaction::prepare_compact() { Defer defer_set_st([&] { if (!st.ok()) { tablet()->set_last_cumu_compaction_status(st.to_string()); - tablet()->set_last_cumu_compaction_failure_time(UnixMillis()); } }); diff --git a/be/src/olap/tablet.cpp b/be/src/olap/tablet.cpp index a2fcc744366172..e54d1be87a530f 100644 --- a/be/src/olap/tablet.cpp +++ b/be/src/olap/tablet.cpp @@ -1796,7 +1796,8 @@ Status Tablet::prepare_compaction_and_calculate_permits( config::enable_sleep_between_delete_cumu_compaction) { tablet->set_last_cumu_compaction_failure_time(UnixMillis()); } - if (!res.is()) { + if (!res.is() && + !res.is()) { DorisMetrics::instance()->cumulative_compaction_request_failed->increment(1); return Status::InternalError("prepare cumulative compaction with err: {}", res.to_string());