fix(planner): Fix materialized view planning when data table has hidden columns (#26545)

## Description

This PR fixes a bug when using the planning-oriented materialized view
framework (introduced in PR #26492) in Hive connector. Unlike the tables
in the Memory connector, the underlying hive tables of the materialized
view contain hidden columns. Therefore, for the comparison with the
number of output variables in the view query, we should use the
underlying data table's visible field count instead of its total field
count.

## Motivation and Context

- Fix a bug when using the planning-oriented materialized view framework
in Hive connector

## Impact

N/A

## Test Plans

- New test case in `TestHiveMaterializedViewLogicalPlanner` which would
fail without this change

## Contributor checklist

- [ ] Please make sure your submission complies with our [contributing
guide](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md),
in particular [code
style](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#code-style)
and [commit
standards](https://github.com/prestodb/presto/blob/master/CONTRIBUTING.md#commit-standards).
- [ ] PR description addresses the issue accurately and concisely. If
the change is non-trivial, a GitHub Issue is referenced.
- [ ] Documented new properties (with its default value), SQL syntax,
functions, or other functionality.
- [ ] If release notes are required, they follow the [release notes
guidelines](https://github.com/prestodb/presto/wiki/Release-Notes-Guidelines).
- [ ] Adequate tests were added if applicable.
- [ ] CI passed.
- [ ] If adding new dependencies, verified they have an [OpenSSF
Scorecard](https://securityscorecards.dev/#the-checks) score of 5.0 or
higher (or obtained explicit TSC approval for lower scores).

## Release Notes

```
== NO RELEASE NOTE ==
```

Author: hantangwangd

presto-hive/src/test/java/com/facebook/presto/hive/TestHiveMaterializedViewLogicalPlanner.java

@@ -72,6 +72,7 @@
 import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.node;
 import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.project;
 import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.singleGroupingSet;
+import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.tableScan;
 import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.unnest;
 import static com.facebook.presto.sql.planner.assertions.PlanMatchPattern.values;
 import static com.facebook.presto.testing.TestingAccessControlManager.TestingPrivilegeType.INSERT_TABLE;
@@ -2921,6 +2922,46 @@ public void testAutoRefreshMaterializedViewAfterInsertion()
         }
     }
 
+    public void testMaterializedViewNotRefreshedInNonLegacyMode()
+    {
+        Session nonLegacySession = Session.builder(getSession())
+                .setSystemProperty("legacy_materialized_views", "false")
+                .build();
+        try {
+            assertUpdate("CREATE TABLE base_table (id BIGINT, name VARCHAR, part_key BIGINT) WITH (partitioned_by = ARRAY['part_key'])");
+            assertUpdate("INSERT INTO base_table VALUES (1, 'Alice', 100), (2, 'Bob', 200)", 2);
+            assertUpdate("CREATE MATERIALIZED VIEW simple_mv WITH (partitioned_by = ARRAY['part_key']) AS SELECT id, name, part_key FROM base_table");
+
+            assertPlan(nonLegacySession, "SELECT * FROM simple_mv",
+                    anyTree(tableScan("base_table")));
+        }
+        finally {
+            assertUpdate("DROP TABLE base_table");
+            assertUpdate("DROP MATERIALIZED VIEW simple_mv");
+        }
+    }
+
+    @Test
+    public void testMaterializedViewRefreshedInNonLegacyMode()
+    {
+        Session nonLegacySession = Session.builder(getSession())
+                .setSystemProperty("legacy_materialized_views", "false")
+                .build();
+        try {
+            assertUpdate("CREATE TABLE base_table (id BIGINT, name VARCHAR, part_key BIGINT) WITH (partitioned_by = ARRAY['part_key'])");
+            assertUpdate("INSERT INTO base_table VALUES (1, 'Alice', 100), (2, 'Bob', 200)", 2);
+            assertUpdate("CREATE MATERIALIZED VIEW simple_mv WITH (partitioned_by = ARRAY['part_key']) AS SELECT id, name, part_key FROM base_table");
+            assertUpdate("REFRESH MATERIALIZED VIEW simple_mv where part_key > 0", 2);
+
+            assertPlan(nonLegacySession, "SELECT * FROM simple_mv",
+                    anyTree(tableScan("simple_mv")));
+        }
+        finally {
+            assertUpdate("DROP TABLE base_table");
+            assertUpdate("DROP MATERIALIZED VIEW simple_mv");
+        }
+    }
+
     private void setReferencedMaterializedViews(DistributedQueryRunner queryRunner, String tableName, List<String> referencedMaterializedViews)
     {
         appendTableParameter(replicateHiveMetastore(queryRunner),

presto-main-base/src/main/java/com/facebook/presto/sql/planner/RelationPlanner.java

@@ -325,10 +325,10 @@ private RelationPlan planMaterializedView(Table node, Analysis.MaterializedViewI
         List<VariableReferenceExpression> viewQueryVariables = viewQueryPlan.getFieldMappings();
 
         checkArgument(
-                dataTableVariables.size() == viewQueryVariables.size(),
-                "Materialized view %s has mismatched field counts: data table has %s fields but view query has %s fields",
+                dataTableDescriptor.getVisibleFieldCount() == viewQueryVariables.size(),
+                "Materialized view %s has mismatched field counts: data table has %s visible fields but view query has %s fields",
                 materializedViewName,
-                dataTableVariables.size(),
+                dataTableDescriptor.getVisibleFieldCount(),
                 viewQueryVariables.size());
 
         ImmutableList.Builder<VariableReferenceExpression> outputVariablesBuilder = ImmutableList.builder();