cloud-fan commented on code in PR #46611:
URL: https://github.com/apache/spark/pull/46611#discussion_r1606151351
##########
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ApplicationMaster.scala:
##########
@@ -696,7 +696,7 @@ private[spark] class ApplicationMaster(
/** Add the Yarn IP filter that is required for properly securing the UI. */
private def addAmIpFilter(driver: Option[RpcEndpointRef], proxyBase: String)
= {
- val amFilter = "org.apache.hadoop.yarn.server.webproxy.amfilter.AmIpFilter"
+ val amFilter = "org.apache.spark.deploy.yarn.AmIpFilter"
Review Comment:
nit: we can use `classOf` to avoid hardcoding the class name here.
##########
streaming/src/main/scala/org/apache/spark/streaming/Checkpoint.scala:
##########
@@ -86,7 +86,7 @@ class Checkpoint(ssc: StreamingContext, val checkpointTime:
Time)
}
// Add Yarn proxy filter specific configurations to the recovered SparkConf
- val filter = "org.apache.hadoop.yarn.server.webproxy.amfilter.AmIpFilter"
+ val filter = "org.apache.spark.deploy.yarn.AmIpFilter"
Review Comment:
ditto
--
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
