MaxGekk commented on code in PR #50590:
URL: https://github.com/apache/spark/pull/50590#discussion_r2044636714
##########
sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala:
##########
@@ -4934,6 +4934,11 @@ class SQLQuerySuite extends QueryTest with
SharedSparkSession with AdaptiveSpark
checkAnswer(df, Row("a"))
}
+
+ test("SPARK-51805: Get with null argument should return null instead of
throwing cast error") {
Review Comment:
Would you mind to move this test to more specific place, for example
`array.sql`
##########
sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala:
##########
@@ -4934,6 +4934,11 @@ class SQLQuerySuite extends QueryTest with
SharedSparkSession with AdaptiveSpark
checkAnswer(df, Row("a"))
}
+
+ test("SPARK-51805: Get with null argument should return null instead of
throwing cast error") {
+ val df = sql("SELECT get(null, 0)")
Review Comment:
Could you check the second argument as `null`, please.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
