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

    https://github.com/apache/spark/pull/19538#discussion_r146111073
  
    --- Diff: core/src/main/scala/org/apache/spark/ui/UIUtils.scala ---
    @@ -506,4 +510,33 @@ private[spark] object UIUtils extends Logging {
     
       def getTimeZoneOffset() : Int =
         TimeZone.getDefault().getOffset(System.currentTimeMillis()) / 1000 / 60
    +
    +  /**
    +  * Return the correct Href after checking if master is running in the
    +  * reverse proxy mode or not.
    +  */
    +  def makeHref(proxy: Boolean, id: String, origHref: String): String = {
    --- End diff --
    
    I think this method came with the original patch.  I don't see anything 
calling it.  I will remove it.


---

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

Reply via email to