[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-20 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-198207419 I think you can actually just make that variable a static int outside the main definition, and -= it in the foreachRDD block. Similar to Scala. I tested it an

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-20 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197732525 Merged build finished. Test PASSed. --- 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 projec

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-20 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-198304584 You'll need to open another one I'm afraid. On Fri, 18 Mar 2016 at 12:50 Ruifeng Zheng wrote: > @MLnick How can I fix it in

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197732446 **[Test build #53399 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53399/consoleFull)** for PR 11776 at commit [`ff56ff5`](https://g

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56482053 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread MLnick
Github user MLnick commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197778301 Merged to master. Thanks! --- 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: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread zhengruifeng
Github user zhengruifeng commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56470345 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,123 @@ +/* + * Licensed to the Ap

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/11776 --- 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 ena

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56478518 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56469940 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,123 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197730304 **[Test build #53399 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53399/consoleFull)** for PR 11776 at commit [`ff56ff5`](https://gi

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197769698 **[Test build #53409 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53409/consoleFull)** for PR 11776 at commit [`f251229`](https://g

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197769822 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56479985 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197757480 Merged build finished. Test PASSed. --- 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 projec

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197757077 **[Test build #53405 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53405/consoleFull)** for PR 11776 at commit [`7750299`](https://g

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197732526 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread zhengruifeng
Github user zhengruifeng commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56468065 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,119 @@ +/* + * Licensed to the Ap

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56476337 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-198153829 @MLnick @srowen What should I do to refine the PR? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-198302793 @MLnick How can I fix it in this PR? Or I have to open another one? --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread zhengruifeng
GitHub user zhengruifeng opened a pull request: https://github.com/apache/spark/pull/11776 [Minor][DOC] Add JavaStreamingTestExample ## What changes were proposed in this pull request? Add the java example of StreamingTest ## How was this patch tested?

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197751636 **[Test build #53405 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53405/consoleFull)** for PR 11776 at commit [`7750299`](https://gi

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread srowen
Github user srowen commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56481486 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197757486 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56481142 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,121 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-19 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197765606 **[Test build #53409 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53409/consoleFull)** for PR 11776 at commit [`f251229`](https://gi

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-18 Thread MLnick
Github user MLnick commented on a diff in the pull request: https://github.com/apache/spark/pull/11776#discussion_r56464958 --- Diff: examples/src/main/java/org/apache/spark/examples/mllib/JavaStreamingTestExample.java --- @@ -0,0 +1,119 @@ +/* + * Licensed to the Apache S

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-18 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-197769816 Merged build finished. Test PASSed. --- 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 projec

[GitHub] spark pull request: [Minor][DOC] Add JavaStreamingTestExample

2016-03-18 Thread zhengruifeng
Github user zhengruifeng commented on the pull request: https://github.com/apache/spark/pull/11776#issuecomment-198309008 @MLnick ok I will open another pr. --- 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