You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by ga...@apache.org on 2026/04/30 02:26:02 UTC

(doris) branch master updated: [Fix](warm up) Fix warm up don't retry due to error message change (#62886)

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

gavinchou 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 412c89ba2c2 [Fix](warm up) Fix warm up don't retry due to error message change (#62886)
412c89ba2c2 is described below

commit 412c89ba2c2553b3a98b45d2530ebf97387c4cd7
Author: bobhan1 <ba...@selectdb.com>
AuthorDate: Thu Apr 30 10:25:54 2026 +0800

    [Fix](warm up) Fix warm up don't retry due to error message change (#62886)
    
    https://github.com/apache/doris/pull/58044 changed the error message
    when tablet is not found in cache but don't change the condition in
    `CloudInternalServiceImpl::warm_up_rowset`. This will cause passive warm
    up don't retry because the status code is not `TABLE_NOT_FOUND`.
    This PR fix it.
---
 be/src/cloud/cloud_internal_service.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/be/src/cloud/cloud_internal_service.cpp b/be/src/cloud/cloud_internal_service.cpp
index a437eb3e778..4d14373edcc 100644
--- a/be/src/cloud/cloud_internal_service.cpp
+++ b/be/src/cloud/cloud_internal_service.cpp
@@ -530,7 +530,8 @@ void CloudInternalServiceImpl::warm_up_rowset(google::protobuf::RpcController* c
                                                    /* local_only = */ local_only);
         if (!res.has_value()) {
             LOG_WARNING("Warm up error ").tag("tablet_id", tablet_id).error(res.error());
-            if (res.error().msg().find("local_only=true") != std::string::npos) {
+            if (res.error().msg().find("local_only=true") != std::string::npos ||
+                res.error().msg().find("force_use_only_cached=true") != std::string::npos) {
                 res.error().set_code(ErrorCode::TABLE_NOT_FOUND);
             }
             res.error().to_protobuf(response->mutable_status());


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org