branch-3.0: [opt](explain) use full qualified name in scan node (#52342) (#52706)

bp #52342

Author: morningman

fe/fe-core/src/main/java/org/apache/doris/catalog/TableIf.java

@@ -488,9 +488,15 @@ default List<String> getFullQualifiers() {
     }
 
     default String getNameWithFullQualifiers() {
-        return String.format("%s.%s.%s", getDatabase().getCatalog().getName(),
-                ClusterNamespace.getNameFromFullName(getDatabase().getFullName()),
-                getName());
+        DatabaseIf db = getDatabase();
+        // Some kind of table like FunctionGenTable does not belong to any database
+        if (db == null) {
+            return "null.null." + getName();
+        } else {
+            return db.getCatalog().getName()
+                    + "." + ClusterNamespace.getNameFromFullName(db.getFullName())
+                    + "." + getName();
+        }
     }
 
     default boolean isManagedTable() {
@@ -522,3 +528,4 @@ default boolean isPartitionedTable() {
 
     TableIndexes getTableIndexes();
 }
+

fe/fe-core/src/main/java/org/apache/doris/datasource/FileScanNode.java

@@ -92,7 +92,7 @@ public long getPushDownCount() {
     @Override
     public String getNodeExplainString(String prefix, TExplainLevel detailLevel) {
         StringBuilder output = new StringBuilder();
-        output.append(prefix).append("table: ").append(desc.getTable().getName()).append("\n");
+        output.append(prefix).append("table: ").append(desc.getTable().getNameWithFullQualifiers()).append("\n");
         if (!conjuncts.isEmpty()) {
             output.append(prefix).append("predicates: ").append(getExplainString(conjuncts)).append("\n");
         }

regression-test/suites/external_table_p0/iceberg/test_iceberg_filter.groovy

@@ -64,6 +64,7 @@ suite("test_iceberg_filter", "p0,external,doris,external_docker,external_docker_
             explain {
                 sql("select * from ${tb_ts_filter} where ts < '2024-05-30 20:34:56'")
                 contains "inputSplitNum=0"
+                contains "table: test_iceberg_filter.multi_catalog.tb_ts_filter"
             }
             explain {
                 sql("select * from ${tb_ts_filter} where ts < '2024-05-30 20:34:56.12'")

regression-test/suites/external_table_p0/tvf/test_tvf_view.groovy

@@ -44,6 +44,7 @@ suite("test_tvf_view", "p0,external,tvf,external_docker,hive") {
             contains("_table_valued_function_hdfs.p_container")
             contains("_table_valued_function_hdfs.p_retailprice")
             contains("_table_valued_function_hdfs.p_comment")
+            contains("table: null.null.HDFSTableValuedFunction")
         }
         explain{
             sql("select * from hdfs (\n" +
@@ -61,7 +62,19 @@ suite("test_tvf_view", "p0,external,tvf,external_docker,hive") {
             contains("_table_valued_function_hdfs.p_comment")
         }
 
-        sql """drop database if exists test_tvf_view_p2"""
+        sql """create view tvf_view_count as select * from hdfs (
+            "uri"="hdfs://${nameNodeHost}:${hdfsPort}/user/doris/tpch1.db/tpch1_parquet/part/part-00000-cb9099f7-a053-4f9a-80af-c659cfa947cc-c000.snappy.parquet",
+            "hadoop.username" = "hadoop",
+            "format"="parquet");"""
+
+        explain {
+            verbose true
+            sql("select count(1) from tvf_view_count")
+            contains "SlotDescriptor{id=0,"
+            notContains "SlotDescriptor{id=1,"
+        }
+
+        // sql """drop database if exists test_tvf_view_p2"""
     }
 }