Fix EXPLAIN (TYPE IO) failure when constraint contains type which cannot be cast to varchar (#27433)

* Avoid exception in EXPLAIN (TYPE IO)

Avoid exception if constraint contain type which can't cast to varchar

* Make castToVarcharOrFail method private in ValuePrinter

* empty: roll the dice 🎲

---------

Co-authored-by: Piotr Findeisen <[email protected]>

Author: hqbhoho

core/trino-main/src/main/java/io/trino/sql/planner/planprinter/IoPlanPrinter.java

@@ -783,7 +783,7 @@ private FormattedDomain parseDomain(Domain domain)
                                     .collect(toImmutableSet())),
                     discreteValues -> formattedRanges.addAll(
                             discreteValues.getValues().stream()
-                                    .map(value -> valuePrinter.castToVarcharOrFail(type, value))
+                                    .map(value -> valuePrinter.castToVarchar(type, value))
                                     .map(value -> new FormattedMarker(Optional.of(value), Bound.EXACTLY))
                                     .map(marker -> new FormattedRange(marker, marker))
                                     .collect(toImmutableSet())),
@@ -799,13 +799,13 @@ private FormattedRange formatRange(Range range)
             FormattedMarker low = range.isLowUnbounded()
                     ? new FormattedMarker(Optional.empty(), Bound.ABOVE)
                     : new FormattedMarker(
-                    Optional.of(valuePrinter.castToVarcharOrFail(range.getType(), range.getLowBoundedValue())),
+                    Optional.of(valuePrinter.castToVarchar(range.getType(), range.getLowBoundedValue())),
                     range.isLowInclusive() ? Bound.EXACTLY : Bound.ABOVE);
 
             FormattedMarker high = range.isHighUnbounded()
                     ? new FormattedMarker(Optional.empty(), Bound.BELOW)
                     : new FormattedMarker(
-                    Optional.of(valuePrinter.castToVarcharOrFail(range.getType(), range.getHighBoundedValue())),
+                    Optional.of(valuePrinter.castToVarchar(range.getType(), range.getHighBoundedValue())),
                     range.isHighInclusive() ? Bound.EXACTLY : Bound.BELOW);
 
             return new FormattedRange(low, high);

core/trino-main/src/main/java/io/trino/sql/planner/planprinter/ValuePrinter.java

@@ -48,7 +48,7 @@ public String castToVarchar(Type type, Object value)
         }
     }
 
-    public String castToVarcharOrFail(Type type, Object value)
+    private String castToVarcharOrFail(Type type, Object value)
             throws OperatorNotFoundException
     {
         if (value == null) {

plugin/trino-hive/src/test/java/io/trino/plugin/hive/BaseHiveConnectorTest.java

@@ -47,6 +47,7 @@
 import io.trino.spi.security.Identity;
 import io.trino.spi.security.SelectedRole;
 import io.trino.spi.statistics.TableStatistics;
+import io.trino.spi.type.ArrayType;
 import io.trino.spi.type.DateType;
 import io.trino.spi.type.TimestampType;
 import io.trino.spi.type.Type;
@@ -1661,6 +1662,33 @@ public void testIoExplainWithPrimitiveTypes()
         }
     }
 
+    @Test
+    public void testIoExplainWithStructuralTypes()
+    {
+        assertUpdate("CREATE TABLE io_explain_test_structural_type(array_col ARRAY(int)) WITH (format='PARQUET')");
+        EstimatedStatsAndCost estimate = new EstimatedStatsAndCost(0.0, 0.0, 0.0, 0.0, 0.0);
+        assertThat(getIoPlanCodec().fromJson((String) getOnlyElement(computeActual("EXPLAIN (TYPE IO, FORMAT JSON) SELECT * FROM io_explain_test_structural_type WHERE array_col = ARRAY[1]").getOnlyColumnAsSet())))
+                .isEqualTo(new IoPlan(
+                        ImmutableSet.of(new TableColumnInfo(
+                                new CatalogSchemaTableName(catalog, "tpch", "io_explain_test_structural_type"),
+                                new IoPlanPrinter.Constraint(
+                                        false,
+                                        ImmutableSet.of(
+                                                new ColumnConstraint(
+                                                        "array_col",
+                                                        new ArrayType(INTEGER),
+                                                        new FormattedDomain(
+                                                                false,
+                                                                ImmutableSet.of(
+                                                                        new FormattedRange(
+                                                                                new FormattedMarker(Optional.of("<UNREPRESENTABLE VALUE>"), EXACTLY),
+                                                                                new FormattedMarker(Optional.of("<UNREPRESENTABLE VALUE>"), EXACTLY))))))),
+                                estimate)),
+                        Optional.empty(),
+                        estimate));
+        assertUpdate("DROP TABLE io_explain_test_structural_type");
+    }
+
     @Test
     public void testReadNoColumns()
     {