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

zykkk 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 db804194471 [hotfix](external) Fixed External meta replay check 
(#47928)
db804194471 is described below

commit db804194471be7455f7c278c9fda24c6f3da6a98
Author: zy-kkk <zhongy...@gmail.com>
AuthorDate: Fri Feb 14 23:24:20 2025 +0800

    [hotfix](external) Fixed External meta replay check (#47928)
    
    Related PR: #47603
    
    Problem Summary:
    
    When checking if a table fetched from meta has a remote name, we should use 
tmpmap
---
 .../main/java/org/apache/doris/datasource/ExternalDatabase.java   | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java 
b/fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java
index ea02a20a07b..6b003f68b5c 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/datasource/ExternalDatabase.java
@@ -237,8 +237,8 @@ public abstract class ExternalDatabase<T extends 
ExternalTable>
             LOG.info("Synchronized table (create): [Name: {}, ID: {}, Remote 
Name: {}]",
                     table.getName(), table.getId(), 
log.getRemoteTableNames().get(i));
         }
-        // Check whether the remoteName and db Tbl db in idToTbl is empty
-        for (T table : idToTbl.values()) {
+        // Check whether the remoteName and db Tbl db in tmpIdToTbl is empty
+        for (T table : tmpIdToTbl.values()) {
             if (Strings.isNullOrEmpty(table.getRemoteName())
                     || table.getDb() == null) {
                 LOG.info("Table [{}] remoteName or database is empty, mark as 
uninitialized",
@@ -664,8 +664,8 @@ public abstract class ExternalDatabase<T extends 
ExternalTable>
                         ((ExternalTable) obj).getName());
             }
         }
-        // Check whether the remoteName and db Tbl db in idToTbl is empty
-        for (T table : idToTbl.values()) {
+        // Check whether the remoteName and db Tbl db in tmpIdToTbl is empty
+        for (T table : tmpIdToTbl.values()) {
             if (Strings.isNullOrEmpty(table.getRemoteName())
                     || table.getDb() == null) {
                 initialized = false;


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

Reply via email to