From b47e1f71b41859724b562903afbae9160c1e9610 Mon Sep 17 00:00:00 2001 From: Aleksandr Efimov Date: Fri, 16 Oct 2026 14:58:44 +0300 Subject: [PATCH] fix(spark): preserve compilation with newer toPrettySQL signatures Spark 4.1 and 4.2 add an optional argument to toPrettySQL. Passing its method reference to map no longer matches the expected unary function. Call it through a lambda so the default argument is applied. --- .../main/scala/io/substrait/spark/logical/ToLogicalPlan.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/spark/src/main/scala/io/substrait/spark/logical/ToLogicalPlan.scala b/spark/src/main/scala/io/substrait/spark/logical/ToLogicalPlan.scala index 6e505fe5f..0435e616b 100644 --- a/spark/src/main/scala/io/substrait/spark/logical/ToLogicalPlan.scala +++ b/spark/src/main/scala/io/substrait/spark/logical/ToLogicalPlan.scala @@ -518,7 +518,7 @@ class ToLogicalPlan(val spark: AnyRef = SparkCompat.instance.getOrCreateSparkSes val hintNames = fieldNames(firstProject) val expressionNames = hintNames .filter(_.size == literals.head.size) - .getOrElse(literals.head.map(toPrettySQL)) + .getOrElse(literals.head.map(literal => toPrettySQL(literal))) val appendedNames = dataSchema.fieldNames.toSeq ++ expressionNames val remappedNames = if (firstProject.getRemap.isPresent) { firstProject.getRemap.get().indices().asScala.map(appendedNames(_)).toSeq