[GitHub] incubator-gearpump pull request #231: [GEARPUMP-355] Fix YarnAppMaster addre...

[manuzhang]

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

    https://github.com/apache/incubator-gearpump/pull/231#discussion_r145602273
  
    --- Diff: 
experiments/yarn/src/main/scala/org/apache/gearpump/experiments/yarn/client/AppMasterResolver.scala
 ---
    @@ -75,3 +64,38 @@ class AppMasterResolver(yarnClient: YarnClient, system: 
ActorSystem) {
         result
       }
     }
    +
    +object AppMasterResolver {
    +
    +  val LOG = LogUtil.getLogger(getClass)
    +
    +  def resolveAppMasterAddress(report: ApplicationReport, system: 
ActorSystem): ActorRef = {
    +    val appMasterPath = s"${report.getOriginalTrackingUrl}" + 
"supervisor-actor-path"
    --- End diff --
    
    missing "/" here


---