Github user jerryshao commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22164#discussion_r213168025
  
    --- Diff: 
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnRMClient.scala
 ---
    @@ -126,4 +136,21 @@ private[spark] class YarnRMClient extends Logging {
         }
       }
     
    +  private def getUrlByRmId(conf: Configuration, rmId: String): String = {
    --- End diff --
    
    For the Spark usage, I think it may not be so useful to use 
`AmFilterInitializer`, because we need to pass the filter parameters to driver 
either from RPC (client mode) or from configuration (cluster mode), in either 
way we should know how to set each parameter, so from my understanding using 
`AmFilterInitializer` seems not so useful. 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to