Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/8340
---
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 enab
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134453479
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134453003
[Test build #41502 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41502/console)
for PR 8340 at commit
[`55fe99e`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134453475
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 project
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37824828
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -426,12 +440,24 @@ class ReceiverTracker(ssc: StreamingC
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134439635
[Test build #41502 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41502/consoleFull)
for PR 8340 at commit
[`55fe99e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134438728
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134438772
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
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37824644
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -426,12 +440,24 @@ class ReceiverTracker(ssc: StreamingC
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37824671
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -484,7 +528,10 @@ class ReceiverTracker(ssc: StreamingCo
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37824577
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -244,24 +244,38 @@ class ReceiverTracker(ssc: StreamingC
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37803312
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -484,7 +528,10 @@ class ReceiverTracker(ssc: StreamingConte
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37802986
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -426,12 +440,24 @@ class ReceiverTracker(ssc: StreamingCont
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37799310
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -426,12 +440,24 @@ class ReceiverTracker(ssc: StreamingCont
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37799147
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -244,24 +244,38 @@ class ReceiverTracker(ssc: StreamingCont
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134346766
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134346765
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134346615
[Test build #41464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41464/console)
for PR 8340 at commit
[`c70690f`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134331463
[Test build #41464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41464/consoleFull)
for PR 8340 at commit
[`c70690f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134330254
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134330214
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 h
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134329128
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134293470
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134293378
[Test build #41455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41455/console)
for PR 8340 at commit
[`c70690f`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134293467
Merged build finished. Test FAILed.
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134258417
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 h
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134262144
[Test build #41455 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41455/consoleFull)
for PR 8340 at commit
[`c70690f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134258513
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134231385
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134231152
[Test build #41452 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41452/console)
for PR 8340 at commit
[`55d9957`](https://github.
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134231386
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134213650
[Test build #41452 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41452/consoleFull)
for PR 8340 at commit
[`55d9957`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134210007
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-134209939
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 h
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37713192
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -431,7 +450,8 @@ class ReceiverTracker(ssc: StreamingContex
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37675815
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -244,24 +244,43 @@ class ReceiverTracker(ssc: StreamingCont
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37675192
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -431,7 +450,8 @@ class ReceiverTracker(ssc: StreamingContex
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133549356
Could you update the main text of this PR and the JIRA to document the main
change? This is a significant change and good to document. Also link this JIRA
with the original
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133478999
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133478995
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133478816
[Test build #41370 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41370/console)
for PR 8340 at commit
[`f0d1f6e`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133462018
[Test build #41370 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41370/consoleFull)
for PR 8340 at commit
[`f0d1f6e`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133460175
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 h
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37644655
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicySuite.scala
---
@@ -127,4 +127,26 @@ class ReceiverScheduling
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37644626
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicy.scala
---
@@ -144,27 +141,31 @@ private[streaming] class Rec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133460250
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
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37600617
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicySuite.scala
---
@@ -127,4 +127,26 @@ class ReceiverSchedulingPol
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37600454
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicySuite.scala
---
@@ -49,16 +49,16 @@ class ReceiverSchedulingPoli
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37599624
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicy.scala
---
@@ -144,27 +141,31 @@ private[streaming] class Rec
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37599615
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicy.scala
---
@@ -144,27 +141,31 @@ private[streaming] class Receiv
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/8340#discussion_r37599461
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverSchedulingPolicy.scala
---
@@ -144,27 +141,31 @@ private[streaming] class Receiv
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133071942
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133071939
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133071757
[Test build #41322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41322/console)
for PR 8340 at commit
[`5fba8d4`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133056489
[Test build #41322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41322/consoleFull)
for PR 8340 at commit
[`5fba8d4`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133054476
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/8340#issuecomment-133054449
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 h
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/8340
[SPARK-10137][Streaming]Avoid to restart receivers if scheduleReceivers
returns balanced results
This PR fixes the following cases for `ReceiverSchedulingPolicy`.
1) Assume there are 4 exec
58 matches
Mail list logo