[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/17 --- 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 enable

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36595932 @ScrapCodes hey sure feel free to submit a follow on PR. I'm going to merge this for now. --- If your project is set up for it, you can reply to this email and have your

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36595569 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36595570 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12984/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread ScrapCodes
Github user ScrapCodes commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36594377 There is one thing to note, `-java-home` currently has a note, we can actually fix that. In the sense by moving check after process args. --- If your project is set up

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread mateiz
Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36594079 Looks good to me too. I read through the updated docs and build instructions. --- If your project is set up for it, you can reply to this email and have your reply appear o

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36593221 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36593222 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36592033 @matiez this looks good to me. Any final thoughts? If not I'll merge this in a bit. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36591694 Jenkins, test this please. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36588452 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36588453 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12977/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36588451 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36588454 One or more automated tests failed Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12978/ --- If your pr

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36586256 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36586257 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36585642 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36585641 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36576064 Hey @ScrapCodes I had some more feedback on documentation and the build stuff. Instead of putting a bunch of comments I just made this pull request into your branch. Do yo

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36576111 I think once those are in this is basically ready to go. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread ScrapCodes
Github user ScrapCodes commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10227199 --- Diff: extras/java8-tests/README.md --- @@ -0,0 +1,15 @@ +# Java 8 test suites. + +These tests are bundled with spark and run if you have java

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36505127 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12970/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36505126 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501482 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501481 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501335 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501336 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12967/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501334 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501338 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12966/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501325 Merged build finished. --- 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 t

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36501327 All automated tests passed. Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/12968/ --- If your project i

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497570 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497569 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497343 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497344 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497236 Merged build triggered. --- 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

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-03 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36497237 Merged build started. --- 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 th

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-02 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36480448 Hey @ScrapCodes I thought about this a bit more - do you mind moving the java8-tests folder into a new folder called `/extras`? We can put things there that aren't include

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10192610 --- Diff: docs/building-with-maven.md --- @@ -76,3 +78,11 @@ The maven build includes support for building a Debian package containing the as $ mvn -P

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on the pull request: https://github.com/apache/spark/pull/17#issuecomment-36436518 Hey @ScrapCodes - this is really cool! It's great that we will get this into Spark 1.0 so that we can support Java 8 lambdas moving forward. I have some high level suggest

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10192306 --- Diff: docs/building-with-maven.md --- @@ -76,3 +78,11 @@ The maven build includes support for building a Debian package containing the as $ mvn -P

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10192279 --- Diff: docs/java-programming-guide.md --- @@ -30,6 +30,12 @@ There are a few key differences between the Java and Scala APIs: classes for key-value p

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10192261 --- Diff: docs/building-with-maven.md --- @@ -76,3 +78,11 @@ The maven build includes support for building a Debian package containing the as $ mvn -P

[GitHub] spark pull request: [java8API] SPARK-964 Investigate the potential...

2014-03-01 Thread pwendell
Github user pwendell commented on a diff in the pull request: https://github.com/apache/spark/pull/17#discussion_r10192134 --- Diff: docs/java-programming-guide.md --- @@ -127,11 +132,20 @@ class Split extends FlatMapFunction { JavaRDD words = lines.flatMap(new Split()); {