ODIn/spark-instrumented-optimizer
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

[SPARK-34940][SQL][TEST] Fix test of BasicWriteTaskStatsTrackerSuite

Browse source 
### 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 <chengsu@fb.com>
Signed-off-by: HyukjinKwon <gurwls223@apache.org>
This commit is contained in:
Cheng Su 2021-04-02 15:51:22 +09:00 committed by HyukjinKwon
parent 1d084513b9
commit 280a2f359c
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/BasicWriteTaskStatsTrackerSuite.scala
View file

@ -149,7 +149,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)