dongjoon-hyun commented on code in PR #45728:
URL: https://github.com/apache/spark/pull/45728#discussion_r1981930953
##########
core/src/main/scala/org/apache/spark/internal/config/UI.scala:
##########
@@ -220,6 +220,18 @@ private[spark] object UI {
.stringConf
.createOptional
+ val CUSTOM_DRIVER_LOG_URL = ConfigBuilder("spark.ui.custom.driver.log.url")
+ .doc("Specifies custom Spark driver log url for supporting external log
service instead of " +
+ "using cluster managers' application log urls in the 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. " +
+ "This configuration 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.")
+ .version("4.0.0")
Review Comment:
Shall we use `4.1.0` since `master` branch is `4.1.0-SNAPSHOT`?
https://github.com/apache/spark/blob/a30bdc3b4c53910d2daa40cbcef782e0e880da9a/pom.xml#L29
--
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
