GitHub user sarutak opened a pull request:

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

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

    @tgravescs reported this issue.
    
    Following is quoted from @tgravescs' 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-2

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

    https://github.com/apache/spark/pull/2981.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 #2981
    
----
commit 70b8882fe48bba6c90f3f0ba15d4d9659c04b1f9
Author: Kousuke Saruta <saru...@oss.nttdata.co.jp>
Date:   2014-10-23T10:36:21Z

    Fixed NPE thrown

commit e2fd6bc5964407cdf09bd86b1c46aba3bb2d1e62
Author: Kousuke Saruta <saru...@oss.nttdata.co.jp>
Date:   2014-10-28T16:41:24Z

    Merge branch 'master' of git://git.apache.org/spark into SPARK-3657

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to