From 92f2068b0801fabd92c76f860e0b06d649a88785 Mon Sep 17 00:00:00 2001 From: Johan Lasperas Date: Mon, 9 Sep 2024 18:49:36 +0200 Subject: [PATCH] [Spark] Fix overflow in GeneratedColumnSuite (#3652) ## Description Small test fix, follow up from https://github.com/delta-io/delta/pull/3601 The test contains an overflow and fails when running with ANSI_MODE enabled. ## How was this patch tested? Test-only --- .../org/apache/spark/sql/delta/GeneratedColumnSuite.scala | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/spark/src/test/scala/org/apache/spark/sql/delta/GeneratedColumnSuite.scala b/spark/src/test/scala/org/apache/spark/sql/delta/GeneratedColumnSuite.scala index 28243d6806..2ff8439d54 100644 --- a/spark/src/test/scala/org/apache/spark/sql/delta/GeneratedColumnSuite.scala +++ b/spark/src/test/scala/org/apache/spark/sql/delta/GeneratedColumnSuite.scala @@ -788,8 +788,8 @@ trait GeneratedColumnSuiteBase test("changing the type of nested field not referenced by a generated col") { withTableName("disallow_column_type_evolution") { table => - createTable(table, None, "t STRUCT, gen SMALLINT", - Map("gen" -> "CAST(HASH(t.a - 10s) AS SMALLINT)"), Nil) + createTable(table, None, "t STRUCT, gen INT", + Map("gen" -> "HASH(t.a)"), Nil) // changing the type of `t.b` should succeed since it is not being // referenced by any CHECK constraints or generated columns. @@ -798,7 +798,7 @@ trait GeneratedColumnSuiteBase .write.format("delta").mode("append") .option("mergeSchema", "true") .saveAsTable(table) - checkAnswer(spark.table(table), Row(Row(32767, 32767), -22677) :: Nil) + checkAnswer(spark.table(table), Row(Row(32767, 32767), 1249274084) :: Nil) } }