Github user sarutak commented on a diff in the pull request:
https://github.com/apache/spark/pull/16625#discussion_r99540244
--- Diff: docs/security.md ---
@@ -49,10 +49,6 @@ component-specific configuration namespaces used to
override the default setting
<th>Component</th>
</tr>
<tr>
- <td><code>spark.ssl.fs</code></td>
--- End diff --
The namespace `fs` seems to be still referred
(https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/SecurityManager.scala#L259
).
We shouldn't remove this description should we?
Also it's chance to modify the description about the namespace in
`SecurityManager.scala`.
```
* All the SSL settings like `spark.ssl.xxx` where `xxx` is a particular
configuration property,
* denote the global configuration for all the supported protocols. In
order to override the global
* configuration for the particular protocol, the properties must be
overwritten in the
* protocol-specific namespace. Use `spark.ssl.yyy.xxx` settings to
overwrite the global
* configuration for particular protocol denoted by `yyy`. Currently `yyy`
can be only`fs` for
* broadcast and file server.
```
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
