(amoro) branch master updated: [AMORO-3065] Optimizing rest api efficiency for table detail (#3260)

[jinsongzhou]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 4524764f1 [AMORO-3065] Optimizing rest api efficiency for table detail 
(#3260)
4524764f1 is described below

commit 4524764f1083c7144e97370346d217914deed48d
Author: Congxian Qiu <qcx978132...@gmail.com>
AuthorDate: Wed Oct 16 18:39:01 2024 +0800

    [AMORO-3065] Optimizing rest api efficiency for table detail (#3260)
    
    For table detail we'll retrive detail info  from underlying table, before 
the change
    there will be two call to the catalog when loading the table: 1) check 
whether the database exists,
    2) load the table for the detail info. After the change, the first call to 
catalog has been removed,
    and use the second call to check if the table exists.
---
 amoro-common/src/main/java/org/apache/amoro/CommonUnifiedCatalog.java | 4 ----
 1 file changed, 4 deletions(-)

diff --git 
a/amoro-common/src/main/java/org/apache/amoro/CommonUnifiedCatalog.java 
b/amoro-common/src/main/java/org/apache/amoro/CommonUnifiedCatalog.java
index 79b6e55be..b49d860fb 100644
--- a/amoro-common/src/main/java/org/apache/amoro/CommonUnifiedCatalog.java
+++ b/amoro-common/src/main/java/org/apache/amoro/CommonUnifiedCatalog.java
@@ -122,10 +122,6 @@ public class CommonUnifiedCatalog implements 
UnifiedCatalog {
 
   @Override
   public AmoroTable<?> loadTable(String database, String table) {
-    if (!databaseExists(database)) {
-      throw new NoSuchDatabaseException("Database: " + database + " does not 
exist.");
-    }
-
     return formatCatalogAsOrder(
             TableFormat.MIXED_HIVE,
             TableFormat.MIXED_ICEBERG,