This is an automated email from the ASF dual-hosted git repository.
maxgekk pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new 1b55fd3ee107 [SPARK-47539][SQL][FOLLOWUP] Fix UT about `variant`
1b55fd3ee107 is described below
commit 1b55fd3ee107ecf41e1716fa8f111a7f5c80ecfc
Author: panbingkun <panbing...@baidu.com>
AuthorDate: Mon Mar 25 20:10:23 2024 +0500
[SPARK-47539][SQL][FOLLOWUP] Fix UT about `variant`
### What changes were proposed in this pull request?
The pr aims to fix `conflict UT` by
https://github.com/apache/spark/pull/45575 `Support to_json(variant)`
Closes #45697
Closes #45698
### Why are the changes needed?
Fix master GA break.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
- Manually test.
<img width="1091" alt="image"
src="https://github.com/apache/spark/assets/15246973/aa52542a-96f4-4f3a-86a5-d4f8b7bc209d";>
./build/sbt "catalyst/testOnly
org.apache.spark.sql.catalyst.expressions.ToPrettyStringSuite"
<img width="516" alt="image"
src="https://github.com/apache/spark/assets/15246973/58a16367-f324-4673-89d8-69e5ea61e32c";>
- Pass GA.
### Was this patch authored or co-authored using generative AI tooling?
No.
Closes #45702 from panbingkun/SPARK-47539_FOLLOWUP.
Authored-by: panbingkun <panbing...@baidu.com>
Signed-off-by: Max Gekk <max.g...@gmail.com>
---
.../apache/spark/sql/catalyst/expressions/ToPrettyStringSuite.scala | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git
a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyStringSuite.scala
b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyStringSuite.scala
index 4e043e72fad3..783fba3bfc0d 100644
---
a/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyStringSuite.scala
+++
b/sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ToPrettyStringSuite.scala
@@ -121,8 +121,7 @@ class ToPrettyStringSuite extends SparkFunSuite with
ExpressionEvalHelper {
}
test("Variant as pretty strings") {
- checkEvaluation(
- ToPrettyString(Literal(new VariantVal(Array[Byte](1, 2, 3),
Array[Byte](4, 5)))),
- UTF8String.fromBytes(Array[Byte](1, 2, 3)).toString)
+ val v = new VariantVal(Array[Byte](1, 2, 3), Array[Byte](1, 1))
+ checkEvaluation(ToPrettyString(Literal(v)),
UTF8String.fromString(v.toString))
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org
