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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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());
{
44 matches
Mail list logo