Optimize materialized_views with schema IN predicate

Before the changes, `system.metadata.materialized_views` queried with
`schema_name IN ...` predicate would first read all materialized views
from all schemas within catalog, and only then filter by schema. This
commit introduces filtering by schema first. For now this is done on the
engine side, but with something like generified `Constraint` class we
could give the connector the opportunity to process the information even
better.

core/trino-main/src/main/java/io/trino/connector/system/MaterializedViewSystemTable.java

@@ -14,6 +14,7 @@
 package io.trino.connector.system;
 
 import com.google.inject.Inject;
+import io.airlift.slice.Slice;
 import io.trino.FullConnectorSession;
 import io.trino.Session;
 import io.trino.metadata.Metadata;
@@ -107,13 +108,23 @@ public RecordCursor cursor(
             return displayTable.build().cursor();
         }
 
-        Optional<String> schemaFilter = tryGetSingleVarcharValue(schemaDomain);
         Optional<String> tableFilter = tryGetSingleVarcharValue(tableDomain);
 
         listCatalogNames(session, metadata, accessControl, catalogDomain).forEach(catalogName -> {
-            QualifiedTablePrefix tablePrefix = tablePrefix(catalogName, schemaFilter, tableFilter);
-
-            addMaterializedViewForCatalog(session, displayTable, tablePrefix);
+            // TODO A connector may be able to pull information from multiple schemas at once, so pass the schema filter to the connector instead.
+            // TODO Support LIKE predicates on schema name (or any other functional predicates), so pass the schema filter as Constraint-like to the connector.
+            if (schemaDomain.isNullableDiscreteSet()) {
+                for (Object slice : schemaDomain.getNullableDiscreteSet().getNonNullValues()) {
+                    String schemaName = ((Slice) slice).toStringUtf8();
+                    if (isImpossibleObjectName(schemaName)) {
+                        continue;
+                    }
+                    addMaterializedViewForCatalog(session, displayTable, tablePrefix(catalogName, Optional.of(schemaName), tableFilter));
+                }
+            }
+            else {
+                addMaterializedViewForCatalog(session, displayTable, tablePrefix(catalogName, Optional.empty(), tableFilter));
+            }
         });
 
         return displayTable.build().cursor();

testing/trino-tests/src/test/java/io/trino/connector/system/metadata/TestSystemMetadataConnector.java

@@ -305,7 +305,8 @@ public void testMaterializedViewsMetadataCalls()
                 // TODO introduce materialized views in CountingMockConnector
                 "VALUES (0, 0, 0, 0)",
                 ImmutableMultiset.<String>builder()
-                        .add("ConnectorMetadata.getMaterializedViews")
+                        .add("ConnectorMetadata.getMaterializedViews(schema=test_schema1)")
+                        .add("ConnectorMetadata.getMaterializedViews(schema=test_schema2)")
                         .build());
 
         // Multiple schemas
@@ -320,7 +321,11 @@ public void testMaterializedViewsMetadataCalls()
                 // TODO introduce materialized views in CountingMockConnector
                 "VALUES (0, 0, 0, 0)",
                 ImmutableMultiset.<String>builder()
-                        .add("ConnectorMetadata.getMaterializedViews")
+                        .add("ConnectorMetadata.getMaterializedViews(schema=test_schema1)")
+                        .add("ConnectorMetadata.getMaterializedViews(schema=test_schema2)")
+                        .addAll(IntStream.range(1, MAX_PREFIXES_COUNT + 1)
+                                .mapToObj("ConnectorMetadata.getMaterializedViews(schema=bogus_schema%s)"::formatted)
+                                .toList())
                         .build());
 
         // Small LIMIT