[GitHub] spark pull request #21422: [Spark-24376][doc]Summary:compiling spark with sc...

2018-05-29 Thread gentlewangyu
Github user gentlewangyu closed the pull request at:

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


---

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



[GitHub] spark pull request #21422: [Spark-24376][doc]Summary:compiling spark with sc...

2018-05-24 Thread gentlewangyu
Github user gentlewangyu commented on a diff in the pull request:

https://github.com/apache/spark/pull/21422#discussion_r190541918
  
--- Diff: docs/building-spark.md ---
@@ -92,10 +92,10 @@ like ZooKeeper and Hadoop itself.
 ./build/mvn -Pmesos -DskipTests clean package
 
 ## Building for Scala 2.10
-To produce a Spark package compiled with Scala 2.10, use the 
`-Dscala-2.10` property:
+To produce a Spark package compiled with Scala 2.10, use the 
`-Pscala-2.10` property:
 
 ./dev/change-scala-version.sh 2.10
-./build/mvn -Pyarn -Dscala-2.10 -DskipTests clean package
+./build/mvn -Pyarn -scala-2.10 -DskipTestsP clean package
--- End diff --

sorry , It's -Pscala-2.10


---

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



[GitHub] spark pull request #21422: [Spark-24376][doc]Summary:compiling spark with sc...

2018-05-24 Thread HyukjinKwon
Github user HyukjinKwon commented on a diff in the pull request:

https://github.com/apache/spark/pull/21422#discussion_r190525438
  
--- Diff: docs/building-spark.md ---
@@ -92,10 +92,10 @@ like ZooKeeper and Hadoop itself.
 ./build/mvn -Pmesos -DskipTests clean package
 
 ## Building for Scala 2.10
-To produce a Spark package compiled with Scala 2.10, use the 
`-Dscala-2.10` property:
+To produce a Spark package compiled with Scala 2.10, use the 
`-Pscala-2.10` property:
 
 ./dev/change-scala-version.sh 2.10
-./build/mvn -Pyarn -Dscala-2.10 -DskipTests clean package
+./build/mvn -Pyarn -scala-2.10 -DskipTestsP clean package
--- End diff --

DskipTestsP?


---

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



[GitHub] spark pull request #21422: [Spark-24376][doc]Summary:compiling spark with sc...

2018-05-24 Thread gentlewangyu
GitHub user gentlewangyu opened a pull request:

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

[Spark-24376][doc]Summary:compiling spark with scala-2.10 should use the -P 
parameter instead of -D



 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/gentlewangyu/spark SPARK-24376

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

https://github.com/apache/spark/pull/21422.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 #21422


commit d5167ed3dd3ce87a8904c2444eaf6c839ea541d3
Author: wangyu 
Date:   2018-05-24T08:06:47Z

git config --global user.email "wan...@cmss.chinamobile.com"

git config --global user.name "gentlewangyu"




---

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