This is an automated email from the ASF dual-hosted git repository.
gurwls223 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 19e68ba34098 [SPARK-45684][SQL][SS][TESTS][FOLLOWUP] Use `++` instead
of `s.c.SeqOps#concat`
19e68ba34098 is described below
commit 19e68ba34098baeef617b31c227a14efc275c46d
Author: yangjie01 <[email protected]>
AuthorDate: Tue Dec 5 08:50:29 2023 +0900
[SPARK-45684][SQL][SS][TESTS][FOLLOWUP] Use `++` instead of
`s.c.SeqOps#concat`
### What changes were proposed in this pull request?
This pr use `++` instead of `s.c.SeqOps#concat` to address comments:
https://github.com/apache/spark/pull/43575#discussion_r1414163363
### Why are the changes needed?
`++` is alias for `concat`, but the readability of ++ is better.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Pass GitHub Actions
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #44161 from LuciferYang/SPARK-45684-FOLLOWUP.
Authored-by: yangjie01 <[email protected]>
Signed-off-by: Hyukjin Kwon <[email protected]>
---
.../src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala
index 883f64ff7af4..cd0bbfd47b2b 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamSuite.scala
@@ -174,7 +174,7 @@ class StreamSuite extends StreamTest {
try {
query.processAllAvailable()
val outputDf = spark.read.parquet(outputDir.getAbsolutePath).as[Long]
- checkDatasetUnorderly[Long](outputDf, (0L to 10L).concat(0L to 10L):
_*)
+ checkDatasetUnorderly[Long](outputDf, (0L to 10L) ++ (0L to 10L): _*)
} finally {
query.stop()
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
