[SPARK-28357][CORE][TEST] Fix Flaky Test - FileAppenderSuite.rolling file appender - size-based rolling

dongjoon-hyun · dongjoon-hyun · commit 46a29fedad87 · 2019-07-11T15:58:22.000-07:00
diff --git a/core/src/test/scala/org/apache/spark/util/FileAppenderSuite.scala b/core/src/test/scala/org/apache/spark/util/FileAppenderSuite.scala
@@ -128,7 +128,7 @@ class FileAppenderSuite extends SparkFunSuite with BeforeAndAfter with Logging {
     val files = testRolling(appender, testOutputStream, textToAppend, 0, isCompressed = true)
     files.foreach { file =>
       logInfo(file.toString + ": " + file.length + " bytes")
-      assert(file.length < rolloverSize)
+      assert(file.length <= rolloverSize)
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -128,7 +128,7 @@ class FileAppenderSuite extends SparkFunSuite with BeforeAndAfter with Logging {`
`128`	`128`	`val files = testRolling(appender, testOutputStream, textToAppend, 0, isCompressed = true)`
`129`	`129`	`files.foreach { file =>`
`130`	`130`	`logInfo(file.toString + ": " + file.length + " bytes")`
`131`		`- assert(file.length < rolloverSize)`
	`131`	`+ assert(file.length <= rolloverSize)`
`132`	`132`	`}`
`133`	`133`	`}`
`134`	`134`