From fe5d0b6a6c7ce7128c2f2c51bfc1e105b68cd8b2 Mon Sep 17 00:00:00 2001
From: Yongqiang YANG <yangyongqiang@selectdb.com>
Date: Mon, 21 Apr 2025 14:26:23 +0800
Subject: [PATCH] [fix](missing version) not catchup is worse than lastFailed
 (#50094)

---
 .../src/main/java/org/apache/doris/catalog/Tablet.java | 10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java b/fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java
index ca7f51da823576..ea76273e6dfbfa 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/Tablet.java
@@ -318,16 +318,14 @@ public List<Replica> getQueryableReplicas(long visibleVersion, Map<Long, Set<Lon
             if (replica.isBad()) {
                 continue;
             }
-
-            if (replica.getLastFailedVersion() > 0) {
-                mayMissingVersionReplica.add(replica);
-                continue;
-            }
-
             if (!replica.checkVersionCatchUp(visibleVersion, false)) {
                 notCatchupReplica.add(replica);
                 continue;
             }
+            if (replica.getLastFailedVersion() > 0) {
+                mayMissingVersionReplica.add(replica);
+                continue;
+            }
 
             Set<Long> thisBeAlivePaths = backendAlivePathHashs.get(replica.getBackendIdWithoutException());
             ReplicaState state = replica.getState();