GitHub user sarutak opened a pull request:

    https://github.com/apache/spark/pull/2728

    [SPARK-3657] yarn alpha YarnRMClientImpl throws NPE 
appMasterRequest.setTrackingUrl starting spark-shell

    @tgraves reported this issue.
    
    Following is quoted from @tgraves' report.
    
    YarnRMClientImpl.registerApplicationMaster can throw null pointer exception 
when setting the trackingurl if its empty:
    
        appMasterRequest.setTrackingUrl(new URI(uiAddress).getAuthority())
    
    
    I hit this just start spark-shell without the tracking url set.
    
    14/09/23 16:18:34 INFO yarn.YarnRMClientImpl: Connecting to ResourceManager 
at kryptonitered-jt1.red.ygrid.yahoo.com/98.139.154.99:8030
    Exception in thread "main" java.lang.NullPointerException
            at 
org.apache.hadoop.yarn.proto.YarnServiceProtos$RegisterApplicationMasterRequestProto$Builder.setTrackingUrl(YarnServiceProtos.java:710)
            at 
org.apache.hadoop.yarn.api.protocolrecords.impl.pb.RegisterApplicationMasterRequestPBImpl.setTrackingUrl(RegisterApplicationMasterRequestPBImpl.java:132)
            at 
org.apache.spark.deploy.yarn.YarnRMClientImpl.registerApplicationMaster(YarnRMClientImpl.scala:102)
            at 
org.apache.spark.deploy.yarn.YarnRMClientImpl.register(YarnRMClientImpl.scala:55)
            at 
org.apache.spark.deploy.yarn.YarnRMClientImpl.register(YarnRMClientImpl.scala:38)
            at 
org.apache.spark.deploy.yarn.ApplicationMaster.registerAM(ApplicationMaster.scala:168)
            at 
org.apache.spark.deploy.yarn.ApplicationMaster.runExecutorLauncher(ApplicationMaster.scala:206)
            at 
org.apache.spark.deploy.yarn.ApplicationMaster.run(ApplicationMaster.scala:120)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/sarutak/spark SPARK-3657

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/2728.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2728
    
----
commit 8b5a96e9ed312ac3936147ce0aea6b37347eedfc
Author: Kousuke Saruta <[email protected]>
Date:   2014-10-09T05:40:14Z

    Fixed NPE thrown when null is passed to setTrackingUrl

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to