EnricoMi commented on code in PR #45464:
URL: https://github.com/apache/spark/pull/45464#discussion_r1519987571
##########
docs/configuration.md:
##########
@@ -1627,15 +1627,13 @@ Apart from these, the following properties are also
available, and may be useful
<td><code>spark.ui.custom.executor.log.url</code></td>
<td>(none)</td>
<td>
- Specifies custom spark executor log URL for supporting external log
service instead of using cluster
+ Specifies custom Spark executor log URL for supporting external log
service instead of using cluster
managers' application log URLs in Spark UI. Spark will support some path
variables via patterns
which can vary on cluster manager. Please check the documentation for your
cluster manager to
see which patterns are supported, if any. <p/>
Please note that this configuration also replaces original log urls in
event log,
which will be also effective when accessing the application on history
server. The new log urls must be
permanent, otherwise you might have dead link for executor log urls.
- <p/>
- For now, only YARN mode supports this configuration
Review Comment:
I think the `Please check the documentation for your cluster manager to see
which patterns are supported, if any.` is sufficient, there is no need to list
which manager supports this conf and which don't. That list easily gets
out-dated.
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
