sabbasani commented on code in PR #13319:
URL: https://github.com/apache/gluten/pull/13319#discussion_r4397767178
##########
gluten-ut/spark41/src/test/scala/org/apache/spark/sql/catalyst/expressions/GlutenMathExpressionsSuite.scala:
##########
@@ -292,13 +292,13 @@ class GlutenMathExpressionsSuite
checkEvaluation(Unhex(Literal("F")), Array[Byte](15))
checkEvaluation(Unhex(Literal("ff")), Array[Byte](-1))
-// checkEvaluation(Unhex(Literal("GG")), null)
+ checkEvaluation(Unhex(Literal("GG")), null)
Review Comment:
Done in d8d36f0e2. spark42 now has the same split: the two lines are
restored in the new unhex returns null for invalid hex input test, and
ClickHouseTestSettings has the matching excludeGlutenTest with the #7232
pointer. Since this touches gluten-ut/spark42/, spark-test-spark42 should now
run on this PR.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
