You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by lizhaoch <gi...@git.apache.org> on 2017/07/24 07:27:38 UTC

[GitHub] spark pull request #18722: JIRA-SPARK-21498-quick start -> one py demo have ...

GitHub user lizhaoch opened a pull request:

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

    JIRA-SPARK-21498-quick start -> one py demo have some bug in code

    I find a bug about 'quick start',and created a new issues,Sean Owen  let
    me to make a pull request, and I do
    
    ## What changes were proposed in this pull request?
    
    (Please fill in changes proposed in this fix)
    
    ## How was this patch tested?
    
    (Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
    (If this patch involves UI changes, please attach a screenshot; otherwise, remove this)
    
    Please review http://spark.apache.org/contributing.html before opening a pull request.


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

    $ git pull https://github.com/lizhaoch/spark master

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

    https://github.com/apache/spark/pull/18722.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 #18722
    
----
commit a928de105a4046741ba59084a7d97a6e6ae56b02
Author: lizhaoch <li...@163.com>
Date:   2017-07-24T07:08:00Z

    JIRA-SPARK-21498-quick start -> one py demo have some bug in code
    
    I find a bug about 'quick start',and created a new issues,Sean Owen  let
    me to make a pull request, and I do

----


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129035611
  
    --- Diff: docs/quick-start.md ---
    @@ -389,7 +389,7 @@ As an example, we'll create a simple Spark application, `SimpleApp.py`:
     from pyspark.sql import SparkSession
     
     logFile = "YOUR_SPARK_HOME/README.md"  # Should be some file on your system
    -spark = SparkSession.builder().appName(appName).master(master).getOrCreate()
    +spark = SparkSession.builder.appName("SimpleApp").getOrCreate() 
    --- End diff --
    
    Not a big deal at all but could we remove the trailing space at the end?


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129209256
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip instal pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
     
     {% highlight bash %}
    -# Use spark-submit to run your application
    +# Use the Python Interpreter to run your application
     $ python SimpleApp.py
     ...
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -
    --- End diff --
    
    Let's revert removing a newline here. Sounds unrelated.


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by lizhaoch <gi...@git.apache.org>.
Github user lizhaoch commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129221078
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip instal pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
     
     {% highlight bash %}
    -# Use spark-submit to run your application
    +# Use the Python Interpreter to run your application
     $ python SimpleApp.py
     ...
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -
    --- End diff --
    
    so many new lines , i did again 


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129248977
  
    --- Diff: docs/quick-start.md ---
    @@ -421,7 +421,7 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark` you can run your application with the regular Python interpreter or use the provided 'spark-submit' as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark`), you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
    --- End diff --
    
    `interpreter` was fine ...


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by lizhaoch <gi...@git.apache.org>.
Github user lizhaoch commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129246403
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark` you can run your application with the regular Python interpreter or use the provided 'spark-submit' as you prefer.
    --- End diff --
    
    submit again ,you are so careful,so good


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

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


[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    @lizhaoch, Could you double check https://github.com/apache/spark/pull/18722#pullrequestreview-51936276? I am repeating the same comments.


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

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

[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    **[Test build #3851 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3851/testReport)** for PR 18722 at commit [`a31cbd8`](https://github.com/apache/spark/commit/a31cbd8aae238e324bcc7de111a7107c89e9a979).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129232934
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
     
     {% highlight bash %}
    -# Use spark-submit to run your application
    +# Use the Python Interpreter to run your application
    --- End diff --
    
    `Interpreter` -> `interpeter`. I think it is Python interpreter.


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

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

[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by lizhaoch <gi...@git.apache.org>.
Github user lizhaoch commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    OK,I have done


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

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

[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129209310
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip instal pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
     
     {% highlight bash %}
    -# Use spark-submit to run your application
    +# Use the Python Interpreter to run your application
    --- End diff --
    
    I don't think we should capitalise "interpreter".


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

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

[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    **[Test build #3851 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3851/testReport)** for PR 18722 at commit [`a31cbd8`](https://github.com/apache/spark/commit/a31cbd8aae238e324bcc7de111a7107c89e9a979).


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129189476
  
    --- Diff: docs/quick-start.md ---
    @@ -389,7 +389,7 @@ As an example, we'll create a simple Spark application, `SimpleApp.py`:
     from pyspark.sql import SparkSession
     
     logFile = "YOUR_SPARK_HOME/README.md"  # Should be some file on your system
    -spark = SparkSession.builder().appName(appName).master(master).getOrCreate()
    +spark = SparkSession.builder.appName("SimpleApp").getOrCreate() 
    --- End diff --
    
    I guess that should be fine.
    
    <img width="450" src="https://user-images.githubusercontent.com/6477701/28550607-12f30adc-711d-11e7-93b5-9fe07738b78a.png">
    
    it is described below and this looks doing the same thing with equivalent Scala / Java examples here.


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

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


[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    **[Test build #3849 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3849/testReport)** for PR 18722 at commit [`77142b5`](https://github.com/apache/spark/commit/77142b51fba82fb67f218fbe44942b6e217d05fe).
     * This patch passes all tests.
     * This patch merges cleanly.
     * This patch adds no public classes.


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

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

[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129244806
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark` you can run your application with the regular Python interpreter or use the provided 'spark-submit' as you prefer.
    --- End diff --
    
    others look good but for the last one ...
    
    `` pip install pyspark` `` ->  `` pip install pyspark`) ``  (missing closing parenthesis).



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

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


[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by SparkQA <gi...@git.apache.org>.
Github user SparkQA commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    **[Test build #3849 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3849/testReport)** for PR 18722 at commit [`77142b5`](https://github.com/apache/spark/commit/77142b51fba82fb67f218fbe44942b6e217d05fe).


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

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

[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129232815
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip install pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
    --- End diff --
    
    `` (e.g., `pip install pyspark` `` -> `` (e.g., `pip install pyspark`) ``


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

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

[GitHub] spark issue #18722: [SPARK-21498][Examples] quick start -> one py demo have ...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    LGTM


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

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

[GitHub] spark issue #18722: JIRA-SPARK-21498-quick start -> one py demo have some bu...

Posted by jerryshao <gi...@git.apache.org>.
Github user jerryshao commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    Can you please fix the PR title, also update the PR description to be more meaningful.


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by HyukjinKwon <gi...@git.apache.org>.
Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129209368
  
    --- Diff: docs/quick-start.md ---
    @@ -421,16 +421,15 @@ $ YOUR_SPARK_HOME/bin/spark-submit \
     Lines with a: 46, Lines with b: 23
     {% endhighlight %}
     
    -If you have PySpark pip installed into your enviroment (e.g. `pip instal pyspark` you can run your application with the regular Python interpeter or use the provided spark-submit as you prefer.
    +If you have PySpark pip installed into your enviroment (e.g., `pip instal pyspark` you can run your application with the regular Python Interpreter or use the provided 'spark-submit' as you prefer.
    --- End diff --
    
    `` (e.g., `pip instal pyspark` `` -> `` `(e.g., `pip install pyspark`) ``


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

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


[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

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


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

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

[GitHub] spark pull request #18722: [SPARK-21498][Examples] quick start -> one py dem...

Posted by felixcheung <gi...@git.apache.org>.
Github user felixcheung commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18722#discussion_r129117923
  
    --- Diff: docs/quick-start.md ---
    @@ -389,7 +389,7 @@ As an example, we'll create a simple Spark application, `SimpleApp.py`:
     from pyspark.sql import SparkSession
     
     logFile = "YOUR_SPARK_HOME/README.md"  # Should be some file on your system
    -spark = SparkSession.builder().appName(appName).master(master).getOrCreate()
    +spark = SparkSession.builder.appName("SimpleApp").getOrCreate() 
    --- End diff --
    
    what about master?


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

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


[GitHub] spark issue #18722: JIRA-SPARK-21498-quick start -> one py demo have some bu...

Posted by AmplabJenkins <gi...@git.apache.org>.
Github user AmplabJenkins commented on the issue:

    https://github.com/apache/spark/pull/18722
  
    Can one of the admins verify this patch?


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

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