Skip to content

Commit ee0e74e

Browse files
DjvuLeesrowen
authored andcommitted
[SPARK-21064][CORE][TEST] Fix the default value bug in NettyBlockTransferServiceSuite
## What changes were proposed in this pull request? The default value for `spark.port.maxRetries` is 100, but we use 10 in the suite file. So we change it to 100 to avoid test failure. ## How was this patch tested? No test Author: DjvuLee <[email protected]> Closes #18280 from djvulee/NettyTestBug. (cherry picked from commit b36ce2a) Signed-off-by: Sean Owen <[email protected]>
1 parent 58a8a37 commit ee0e74e

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

core/src/test/scala/org/apache/spark/network/netty/NettyBlockTransferServiceSuite.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -81,7 +81,8 @@ class NettyBlockTransferServiceSuite
8181
private def verifyServicePort(expectedPort: Int, actualPort: Int): Unit = {
8282
actualPort should be >= expectedPort
8383
// avoid testing equality in case of simultaneous tests
84-
actualPort should be <= (expectedPort + 10)
84+
// the default value for `spark.port.maxRetries` is 100 under test
85+
actualPort should be <= (expectedPort + 100)
8586
}
8687

8788
private def createService(port: Int): NettyBlockTransferService = {

0 commit comments

Comments
 (0)