Repository: spark
Updated Branches:
  refs/heads/branch-2.3 d0f30e3f3 -> 54aeae7ba


[MINOR] Add port SSL config in toString and scaladoc

## What changes were proposed in this pull request?

SPARK-17874 introduced a new configuration to set the port where SSL services 
bind to. We missed to update the scaladoc and the `toString` method, though. 
The PR adds it in the missing places

## How was this patch tested?

checked the `toString` output in the logs

Author: Marco Gaido <marcogaid...@gmail.com>

Closes #21429 from mgaido91/minor_ssl.

(cherry picked from commit fd315f5884c03c6dd21abca178897584dee83f1a)
Signed-off-by: Marcelo Vanzin <van...@cloudera.com>


Project: http://git-wip-us.apache.org/repos/asf/spark/repo
Commit: http://git-wip-us.apache.org/repos/asf/spark/commit/54aeae7b
Tree: http://git-wip-us.apache.org/repos/asf/spark/tree/54aeae7b
Diff: http://git-wip-us.apache.org/repos/asf/spark/diff/54aeae7b

Branch: refs/heads/branch-2.3
Commit: 54aeae7ba7c2bdf2badd98c3679e8978c2c38d47
Parents: d0f30e3
Author: Marco Gaido <marcogaid...@gmail.com>
Authored: Fri May 25 12:49:06 2018 -0700
Committer: Marcelo Vanzin <van...@cloudera.com>
Committed: Fri May 25 12:49:19 2018 -0700

----------------------------------------------------------------------
 core/src/main/scala/org/apache/spark/SSLOptions.scala | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/spark/blob/54aeae7b/core/src/main/scala/org/apache/spark/SSLOptions.scala
----------------------------------------------------------------------
diff --git a/core/src/main/scala/org/apache/spark/SSLOptions.scala 
b/core/src/main/scala/org/apache/spark/SSLOptions.scala
index 477b019..04c38f1 100644
--- a/core/src/main/scala/org/apache/spark/SSLOptions.scala
+++ b/core/src/main/scala/org/apache/spark/SSLOptions.scala
@@ -128,7 +128,7 @@ private[spark] case class SSLOptions(
   }
 
   /** Returns a string representation of this SSLOptions with all the 
passwords masked. */
-  override def toString: String = s"SSLOptions{enabled=$enabled, " +
+  override def toString: String = s"SSLOptions{enabled=$enabled, port=$port, " 
+
       s"keyStore=$keyStore, keyStorePassword=${keyStorePassword.map(_ => 
"xxx")}, " +
       s"trustStore=$trustStore, trustStorePassword=${trustStorePassword.map(_ 
=> "xxx")}, " +
       s"protocol=$protocol, enabledAlgorithms=$enabledAlgorithms}"
@@ -142,6 +142,7 @@ private[spark] object SSLOptions extends Logging {
    *
    * The following settings are allowed:
    * $ - `[ns].enabled` - `true` or `false`, to enable or disable SSL 
respectively
+   * $ - `[ns].port` - the port where to bind the SSL server
    * $ - `[ns].keyStore` - a path to the key-store file; can be relative to 
the current directory
    * $ - `[ns].keyStorePassword` - a password to the key-store file
    * $ - `[ns].keyPassword` - a password to the private key


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@spark.apache.org
For additional commands, e-mail: commits-h...@spark.apache.org

Reply via email to