Skip to content

Commit 0803ec2

Browse files
committed
Review comment updates - remove extra newline, simplify assert in test
1 parent 658a8ec commit 0803ec2

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

core/src/main/scala/org/apache/spark/ui/SparkUI.scala

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,6 @@ private[spark] class SparkUI(
119119

120120
private[spark] def appUIAddress = "http://" + publicHost + ":" + boundPort
121121

122-
123122
}
124123

125124
private[spark] object SparkUI {

core/src/test/scala/org/apache/spark/SparkUISuite.scala

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,12 +25,11 @@ class SparkUISuite extends FunSuite with SharedSparkContext {
2525

2626
test("verify appUIHostPort doesn't contain scheme") {
2727
val appUIUri = new URI(sc.ui.appUIHostPort)
28-
assert(appUIUri.getScheme().startsWith("http") == false)
28+
assert(!appUIUri.getScheme().startsWith("http"))
2929
}
3030

3131
test("verify appUIAddress contains the scheme") {
3232
val appUIUri = new URI(sc.ui.appUIAddress)
33-
assert(appUIUri.getScheme().startsWith("http") == true)
33+
assert(appUIUri.getScheme().startsWith("http"))
3434
}
35-
3635
}

0 commit comments

Comments
 (0)