[GitHub] [incubator-doris] dujl commented on a diff in pull request #9752: [feature](hudi) Step2: Support query hudi external table(include cow and mor table) #9559

[GitBox <gi...@apache.org>]

dujl commented on code in PR #9752:
URL: https://github.com/apache/incubator-doris/pull/9752#discussion_r882576583


##########
fe/fe-core/src/main/java/org/apache/doris/analysis/Analyzer.java:
##########
@@ -626,6 +628,11 @@ public TableRef resolveTableRef(TableRef tableRef) throws AnalysisException {
             }
         }
 
+        if (table.getType() == TableType.HUDI && table.getFullSchema().isEmpty()) {
+            // resolve hudi table's schema when table schema is empty from doris meta

Review Comment:
   two case:
   1. create hudi table with schema. when table schema has been changed in HMS，all hudi table's schema are not lost in HMS，query successful. if some field has been renamed or drop in HMS, query failed.
   2. create a hudi table without schema. when table schema has been changed in HMS, it will resolve schema from hms when query. 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


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