This is an automated email from the ASF dual-hosted git repository.
gurwls223 pushed a commit to branch branch-3.1
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/branch-3.1 by this push:
new 12d39d5 [SPARK-34940][SQL][TEST] Fix test of
BasicWriteTaskStatsTrackerSuite
12d39d5 is described below
commit 12d39d52d98152cd2a95a8fa358a59eaf56b7ab5
Author: Cheng Su <[email protected]>
AuthorDate: Fri Apr 2 15:51:22 2021 +0900
[SPARK-34940][SQL][TEST] Fix test of BasicWriteTaskStatsTrackerSuite
### What changes were proposed in this pull request?
This is to fix the minor typo in unit test of
BasicWriteTaskStatsTrackerSuite
(https://github.com/apache/spark/blob/master/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala#L152
), where it should be a new file name, e.g. `f-3-3`, because the unit test
expects 3 files in statistics
(https://github.com/apache/spark/blob/master/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala#L160
).
### Why are the changes needed?
Fix minor bug.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Changed unit test `"Three files, last one empty"` itself.
Closes #32034 from c21/tracker-fix.
Authored-by: Cheng Su <[email protected]>
Signed-off-by: HyukjinKwon <[email protected]>
(cherry picked from commit 280a2f359c55d8c0563ebbf54a3196ffeb65560e)
Signed-off-by: HyukjinKwon <[email protected]>
---
.../sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala
index 32941d8..5756bed 100644
---
a/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala
+++
b/sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala
@@ -148,7 +148,7 @@ class BasicWriteTaskStatsTrackerSuite extends SparkFunSuite
{
test("Three files, last one empty") {
val file1 = new Path(tempDirPath, "f-3-1")
val file2 = new Path(tempDirPath, "f-3-2")
- val file3 = new Path(tempDirPath, "f-3-2")
+ val file3 = new Path(tempDirPath, "f-3-3")
val tracker = new BasicWriteTaskStatsTracker(conf)
tracker.newFile(file1.toString)
write1(file1)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
