Review comment updates - remove extra newline, simplify assert in test

apache · Apr 7, 2014 · 0803ec2 · 0803ec2
1 parent 658a8ec
commit 0803ec2
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/core/src/main/scala/org/apache/spark/ui/SparkUI.scala b/core/src/main/scala/org/apache/spark/ui/SparkUI.scala
@@ -119,7 +119,6 @@ private[spark] class SparkUI(
 
   private[spark] def appUIAddress = "http://" + publicHost + ":" + boundPort
 
-
 }
 
 private[spark] object SparkUI {

diff --git a/core/src/test/scala/org/apache/spark/SparkUISuite.scala b/core/src/test/scala/org/apache/spark/SparkUISuite.scala
@@ -25,12 +25,11 @@ class SparkUISuite extends FunSuite with SharedSparkContext {
 
   test("verify appUIHostPort doesn't contain scheme") {
     val appUIUri = new URI(sc.ui.appUIHostPort)
-    assert(appUIUri.getScheme().startsWith("http")  == false)
+    assert(!appUIUri.getScheme().startsWith("http"))
   }
 
   test("verify appUIAddress contains the scheme") {
     val appUIUri = new URI(sc.ui.appUIAddress)
-    assert(appUIUri.getScheme().startsWith("http")  == true)
+    assert(appUIUri.getScheme().startsWith("http"))
   }
-
 }
Original file line number	Diff line number	Diff line change
Expand Up		@@ -119,7 +119,6 @@ private[spark] class SparkUI(

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


		}

		private[spark] object SparkUI {
Expand Down