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

    https://github.com/apache/spark/pull/2014#discussion_r17458463
  
    --- Diff: README.md ---
    @@ -13,16 +13,19 @@ and Spark Streaming.
     ## Online Documentation
     
     You can find the latest Spark documentation, including a programming
    -guide, on the project webpage at 
<http://spark.apache.org/documentation.html>.
    +guide, on the [project web 
page](http://spark.apache.org/documentation.html).
     This README file only contains basic setup instructions.
     
     ## Building Spark
     
    -Spark is built on Scala 2.10. To build Spark and its example programs, run:
    +Spark is built using [Apache Maven](http://maven.apache.org/).
    +To build Spark and its example programs, run:
     
    -    ./sbt/sbt assembly
    +    mvn -DskipTests clean package
    --- End diff --
    
    Actually the officially documented way of building Spark is through maven: 
http://spark.apache.org/docs/latest/building-with-maven.html. We should keep 
this consistent with the docs. (The discussion you linked to refers to tests).


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