shrprasa commented on code in PR #39221:
URL: https://github.com/apache/spark/pull/39221#discussion_r1060287776
##########
core/src/test/scala/org/apache/spark/SSLOptionsSuite.scala:
##########
@@ -109,7 +109,7 @@ class SSLOptionsSuite extends SparkFunSuite {
val conf = new SparkConf
val hadoopConf = new Configuration()
conf.set("spark.ssl.enabled", "true")
- conf.set("spark.ssl.ui.enabled", "false")
+ conf.set("spark.ssl.ui.enabled", "true")
Review Comment:
This change is required because the test case is validating the values set
at sub settings (spark.ssl.ui.*) even though spark.ssl.ui.enabled is disabled.
Before this change these values were set but now they won't be set. For this to
work, spark.ssl.ui.enabled should be set to true.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
