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:
@LuciferYang 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]
