Github user srowen commented on the pull request:

    https://github.com/apache/spark/pull/926#issuecomment-44753736
  
    I'm just copying-and-pasting to get something similar running externally. 
Maybe it's a little surprising that the example code doesn't work that way -- 
being in a `main()` kind of suggests this is a stand-alone program. Maybe just 
me. 
    
    I think there are a few possibilities:
    
    - Change all example code to set master if missing (that's what the current 
PR does)
    - Change SparkConf to do something similar as a global default
    - Just update javadoc to make it clear that the examples require the 
`spark.master` system property to be set
    
    I slightly prefer one of the first two on the principle of least surprise, 
but can go any direction. I think at least the third should be done. What say 
everyone?


---
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.
---

Reply via email to