Skip to content

Commit

Permalink
[SPARK-13601] [TESTS] use 1 partition in tests to avoid race conditions
Browse files Browse the repository at this point in the history
## What changes were proposed in this pull request?

Fix race conditions when cleanup files.

## How was this patch tested?

Existing tests.

Author: Davies Liu <[email protected]>

Closes #11507 from davies/flaky.
  • Loading branch information
Davies Liu authored and davies committed Mar 4, 2016
1 parent b373a88 commit d062587
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ class CommitFailureTestRelationSuite extends SQLTestUtils with TestHiveSingleton
withTempPath { file =>
// fail the job in the middle of writing
val divideByZero = udf((x: Int) => { x / (x - 1)})
val df = sqlContext.range(0, 10).select(divideByZero(col("id")))
val df = sqlContext.range(0, 10).coalesce(1).select(divideByZero(col("id")))

SimpleTextRelation.callbackCalled = false
intercept[SparkException] {
Expand All @@ -69,7 +69,8 @@ class CommitFailureTestRelationSuite extends SQLTestUtils with TestHiveSingleton
withTempPath { file =>
// fail the job in the middle of writing
val divideByZero = udf((x: Int) => { x / (x - 1)})
val df = sqlContext.range(0, 10).select(col("id").mod(2).as("key"), divideByZero(col("id")))
val df = sqlContext.range(0, 10).coalesce(1)
.select(col("id").mod(2).as("key"), divideByZero(col("id")))

SimpleTextRelation.callbackCalled = false
intercept[SparkException] {
Expand All @@ -87,7 +88,7 @@ class CommitFailureTestRelationSuite extends SQLTestUtils with TestHiveSingleton
SimpleTextRelation.failCommitter = false
withTempPath { file =>
// fail the job in the middle of writing
val df = sqlContext.range(0, 10).select(col("id").mod(2).as("key"), col("id"))
val df = sqlContext.range(0, 10).coalesce(1).select(col("id").mod(2).as("key"), col("id"))

SimpleTextRelation.callbackCalled = false
SimpleTextRelation.failWriter = true
Expand Down

0 comments on commit d062587

Please sign in to comment.