diff --git a/sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala b/sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala
index c4bf4bea90948..bbc396c879de1 100644
--- a/sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala
+++ b/sql/core/src/test/scala/org/apache/spark/sql/SparkSessionBuilderSuite.scala
@@ -330,10 +330,8 @@ class SparkSessionBuilderSuite extends SparkFunSuite with Eventually {
       .set(wh, "./data1")
       .set(td, "bob")
 
-    // This creates an active SparkContext, which will be picked up the session.
+    // This creates an active SparkContext, which will be picked up by the session below.
     new SparkContext(conf)
-
-    // The only way this could have ever worked if this is effectively a no-op.
     val spark = SparkSession.builder()
       .config(wh, "./data2")
       .config(td, "alice")