[BugFix] Key MV pinned-range map by Table#getUUID() so it is cross-database-unique

The pinned-range map is keyed by a table identifier that producers
(setupPinnedRangesIfNeeded) and consumers (MVPCTRefreshPlanBuilder,
PartitionDiffer#pinnedRangeFor) both compute. Using Table#getTableIdentifier()
keys native tables by their bare name, which is not unique across databases: an
MV referencing two same-named tables in different databases overwrites one
entry, so one table's pinned snapshot is lost.

Key the map by Table#getUUID() on both sides instead — it is cross-database-unique
for every table kind (the table id for native/cloud-native, catalog.db.table.uuid
for external) and is identical at producer and every consumer.
IcebergTable#getUUID() also gains an instanceof-BaseTable guard so a non-BaseTable
native table degrades to an empty uuid instead of a ClassCastException, matching
getTableIdentifier().

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Signed-off-by: Youngwb <yangwenbo_mailbox@163.com>

Author: Youngwb

fe/fe-core/src/main/java/com/starrocks/catalog/IcebergTable.java

@@ -173,7 +173,9 @@ public String getCatalogTableName() {
     @Override
     public String getUUID() {
         if (CatalogMgr.isExternalCatalog(catalogName)) {
-            String uuid = ((BaseTable) getNativeTable()).operations().current().uuid();
+            org.apache.iceberg.Table nativeTable = getNativeTable();
+            String uuid = nativeTable instanceof BaseTable
+                    ? ((BaseTable) nativeTable).operations().current().uuid() : null;
             return String.join(".", catalogName, catalogDBName, catalogTableName,
                     uuid == null ? "" : uuid);
         } else {

fe/fe-core/src/main/java/com/starrocks/scheduler/mv/MVRefreshProcessor.java

@@ -340,10 +340,10 @@ protected void setupPinnedRangesIfNeeded() {
                 continue;
             }
             final TvrVersionRange pinned = TvrTableSnapshot.of(tvr.end());
-            // Must key by Table#getTableIdentifier() to match every consumer (MVPCTRefreshPlanBuilder,
-            // PartitionDiffer#pinnedRangeFor). For native OLAP BaseTableInfo#getTableIdentifier() yields
-            // the table id but Table yields the name, so the old key silently missed every lookup.
-            pinnedMap.put(info.getBaseTable().getTableIdentifier(), pinned);
+            // Key by getUUID() — cross-db-unique and identical at every consumer
+            // (MVPCTRefreshPlanBuilder, PartitionDiffer#pinnedRangeFor). A bare table
+            // name would collide across databases.
+            pinnedMap.put(info.getBaseTable().getUUID(), pinned);
 
             if (info instanceof PCTTableSnapshotInfo) {
                 ((PCTTableSnapshotInfo) info).setPinnedRange(pinned);

fe/fe-core/src/main/java/com/starrocks/scheduler/mv/pct/MVPCTRefreshPlanBuilder.java

@@ -142,12 +142,12 @@ private InsertStmt buildInsertPlan(InsertStmt insertStmt,
         for (Map.Entry<String, TableRelation> entry : tableRelations.entries()) {
             TableRelation relation = entry.getValue();
             Table table = relation.getTable();
-            // Skip null here to avoid NPE on getTableIdentifier(); the relation-grouping loop below
+            // Skip null here to avoid NPE on getUUID(); the relation-grouping loop below
             // throws a controlled AnalysisException for any base table that resolved to null.
             if (table == null) {
                 continue;
             }
-            TvrVersionRange pinned = pinnedMap.get(table.getTableIdentifier());
+            TvrVersionRange pinned = pinnedMap.get(table.getUUID());
             if (pinned != null) {
                 relation.setTvrVersionRange(pinned);
                 logger.info("Inject pinned TVR {} into table relation {} for scan",

fe/fe-core/src/main/java/com/starrocks/sql/common/PartitionDiffer.java

@@ -35,7 +35,7 @@ public abstract class PartitionDiffer {
     // consider partition_ttl_number and mv refresh will consider it to avoid creating too much partitions
     protected final MVTimelinessArbiter.QueryRewriteParams queryRewriteParams;
 
-    // Pinned ranges keyed by Table.getTableIdentifier(). Empty means no pinning.
+    // Pinned ranges keyed by Table.getUUID(). Empty means no pinning.
     protected Map<String, TvrVersionRange> pinnedRangeByTableIdentifier = Collections.emptyMap();
 
     public PartitionDiffer(MaterializedView mv, MVTimelinessArbiter.QueryRewriteParams queryRewriteParams) {
@@ -49,7 +49,7 @@ public void setPinnedRanges(Map<String, TvrVersionRange> pinnedRangeByTableIdent
     }
 
     protected TvrVersionRange pinnedRangeFor(Table table) {
-        return pinnedRangeByTableIdentifier.get(table.getTableIdentifier());
+        return pinnedRangeByTableIdentifier.get(table.getUUID());
     }
 
     /**