Fix handling of dropped base tables for Iceberg MVs

Because the engine does not consider MVs stale while they are still
within the grace period, an error should not be returned even if a base
table has been dropped and the grace period has not yet elapsed.

Author: piotrrzysko

plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/IcebergMetadata.java

@@ -3928,13 +3928,10 @@ case FirstChangeSnapshot(Snapshot snapshot) -> {
                     firstTableChange = firstTableChange
                             .map(epochMilli -> Math.min(epochMilli, snapshot.timestampMillis()));
                 }
-                case UnknownTableChange() -> {
+                case UnknownTableChange(), GoneOrCorruptedTableChange() -> {
                     hasStaleIcebergTables = true;
                     firstTableChange = Optional.empty();
                 }
-                case GoneOrCorruptedTableChange() -> {
-                    return new MaterializedViewFreshness(STALE, Optional.empty());
-                }
             }
         }
 

testing/trino-testing/src/main/java/io/trino/testing/BaseConnectorTest.java

@@ -1801,9 +1801,14 @@ private void testMaterializedViewBaseTableGone(boolean initialized)
             assertUpdate("REFRESH MATERIALIZED VIEW " + viewName, 1);
         }
         assertUpdate("DROP TABLE " + baseTable);
-        assertQueryFails(
-                "TABLE " + viewName,
-                "line 1:1: Failed analyzing stored view '%1$s\\.%2$s\\.%3$s': line 3:3: Table '%1$s\\.%2$s\\.%4$s' does not exist".formatted(catalog, schema, viewName, baseTable));
+        if (initialized) {
+            assertQuerySucceeds("TABLE " + viewName);
+        }
+        else {
+            assertQueryFails(
+                    "TABLE " + viewName,
+                    "line 1:1: Failed analyzing stored view '%1$s\\.%2$s\\.%3$s': line 3:3: Table '%1$s\\.%2$s\\.%4$s' does not exist".formatted(catalog, schema, viewName, baseTable));
+        }
         assertUpdate("DROP MATERIALIZED VIEW " + viewName);
     }