From c70e62c709ad3eb09b0ee33db4d30d26dbe37879 Mon Sep 17 00:00:00 2001 From: Kaijie Chen Date: Wed, 24 Sep 2025 11:40:22 +0800 Subject: [PATCH] [metrics](warmup) add bvar for skipped warmup rowsets (#56301) ### What problem does this PR solve? Problem Summary: Adds the bvar `file_cache_event_driven_warm_up_skipped_rowset_num` to the source cluster Backend to improve observability of the cache warmup process. During event-driven warmup, rowsets can be skipped if the Backend fails to find the tablet's replica location information from a `TGetTabletReplicaInfosRequest`. This new metric makes it possible to monitor and alert on these events, which helps in diagnosing incomplete cache warmups. --- be/src/cloud/cloud_warm_up_manager.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/be/src/cloud/cloud_warm_up_manager.cpp b/be/src/cloud/cloud_warm_up_manager.cpp index d536a641a89b31..2915b81f2536a3 100644 --- a/be/src/cloud/cloud_warm_up_manager.cpp +++ b/be/src/cloud/cloud_warm_up_manager.cpp @@ -40,6 +40,8 @@ namespace doris { +bvar::Adder g_file_cache_event_driven_warm_up_skipped_rowset_num( + "file_cache_event_driven_warm_up_skipped_rowset_num"); bvar::Adder g_file_cache_event_driven_warm_up_requested_segment_size( "file_cache_event_driven_warm_up_requested_segment_size"); bvar::Adder g_file_cache_event_driven_warm_up_requested_segment_num( @@ -507,6 +509,7 @@ void CloudWarmUpManager::warm_up_rowset(RowsetMeta& rs_meta, int64_t sync_wait_t if (replicas.empty()) { VLOG_DEBUG << "There is no need to warmup tablet=" << rs_meta.tablet_id() << ", skipping rowset=" << rs_meta.rowset_id().to_string(); + g_file_cache_event_driven_warm_up_skipped_rowset_num << 1; return; } Status st = _do_warm_up_rowset(rs_meta, replicas, sync_wait_timeout_ms, !cache_hit);