This is an automated email from the ASF dual-hosted git repository.

dataroaring pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/master by this push:
     new ad9138b2bd1 [fix](warmup) ensure warmup only triggers if tablet exists 
on target backend (#53852)
ad9138b2bd1 is described below

commit ad9138b2bd157710ac95b46013d88e30003da375
Author: Kaijie Chen <[email protected]>
AuthorDate: Mon Aug 11 21:59:04 2025 +0800

    [fix](warmup) ensure warmup only triggers if tablet exists on target 
backend (#53852)
    
    ### What problem does this PR solve?
    
    Problem Summary:
    
    For event-driven warmup jobs, ensure warmup only triggers if tablet
    exists on the target backend.
    Because the source BE will cache the tablet location for a short time,
    after rebalancing, the old target BE will still receive warm up requests
    during that time.
    This PR prevents the old target BE to re-fetch the tablet cache when it
    no longer holds the tablet.
    For the new target BE, it will take a different path
    (warm_up_cache_async) to do the warm up.
---
 be/src/cloud/cloud_internal_service.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/be/src/cloud/cloud_internal_service.cpp 
b/be/src/cloud/cloud_internal_service.cpp
index fd0c98bb028..5641bc1ac54 100644
--- a/be/src/cloud/cloud_internal_service.cpp
+++ b/be/src/cloud/cloud_internal_service.cpp
@@ -188,7 +188,10 @@ void 
CloudInternalServiceImpl::warm_up_rowset(google::protobuf::RpcController* c
             continue;
         }
         int64_t tablet_id = rs_meta.tablet_id();
-        auto res = _engine.tablet_mgr().get_tablet(tablet_id);
+        auto res = _engine.tablet_mgr().get_tablet(tablet_id, /* warmup_data = 
*/ false,
+                                                   /* sync_delete_bitmap = */ 
true,
+                                                   /* sync_stats = */ nullptr,
+                                                   /* local_only = */ true);
         if (!res.has_value()) {
             LOG_WARNING("Warm up error ").tag("tablet_id", 
tablet_id).error(res.error());
             continue;


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to