Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121956439
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/6294#issuecomment-121956389
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/6294#issuecomment-121875047
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121875036
[Test build #37482 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37482/console)
for PR 6294 at commit
[`b4c29e7`](https://github.
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121874151
[Test build #37482 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/37482/consoleFull)
for PR 6294 at commit
[`b4c29e7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121873958
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/6294#issuecomment-121873936
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 reopened a pull request:
https://github.com/apache/spark/pull/6294
[SPARK-5681][Streaming] Use a lock to eliminate the race condition when
stopping receivers and registering receivers happen at the same time
This is an alternative way to fix `SPARK-5681`. It mini
Github user zsxwing closed the pull request at:
https://github.com/apache/spark/pull/6294
---
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 user tdas commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121395535
I think this is not needed any more. We should merge this changes into
#7276 as that touches this significantly. Please update the title in that PR to
include this JIRA and
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-121072628
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 the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-117215268
> Yes, but @zsxwing does this affect the receiver management stuff you are
> designing?
Right. Let's wait until we finalize the receiver management design.
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-116847847
Yes, but @zsxwing does this affect the receiver management stuff you are
designing?
On Mon, Jun 29, 2015 at 2:07 PM, andrewor14
wrote:
> @zsxwing <
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-116846507
@zsxwing @tdas the hanging issue seems like one that would be good to fix.
Any updates on this patch?
---
If your project is set up for it, you can reply to this ema
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-105205521
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-105205519
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/6294#issuecomment-105205508
[Test build #33465 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33465/consoleFull)
for PR 6294 at commit
[`b4c29e7`](https://git
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/6294#discussion_r30970195
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
---
@@ -136,12 +141,19 @@ private[streaming] abstract class R
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-105173571
[Test build #33465 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33465/consoleFull)
for PR 6294 at commit
[`b4c29e7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-105172926
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/6294#issuecomment-105172981
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/6294#issuecomment-104784629
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/6294#issuecomment-104784617
[Test build #33359 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33359/consoleFull)
for PR 6294 at commit
[`c41ee94`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104784631
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104760742
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/6294#issuecomment-104760731
[Test build #33353 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33353/consoleFull)
for PR 6294 at commit
[`a8120c0`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104760744
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104756812
[Test build #33359 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33359/consoleFull)
for PR 6294 at commit
[`c41ee94`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104756439
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 the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104756506
@tdas updated as per our discussion offline.
---
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 proj
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104756421
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/6294#discussion_r30921536
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -82,13 +113,23 @@ class ReceiverTracker(ssc: StreamingConte
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741810
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 does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741802
[Test build #33351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33351/consoleFull)
for PR 6294 at commit
[`7b1d9af`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741812
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741740
> Merge conflicts in this PR.
Fixed
---
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 proj
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741592
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741654
[Test build #33353 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33353/consoleFull)
for PR 6294 at commit
[`a8120c0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104741570
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/6294#discussion_r30920425
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -82,13 +113,23 @@ class ReceiverTracker(ssc: StreamingCo
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104740084
Merge conflicts in this 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 does not have th
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104739924
[Test build #33351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33351/consoleFull)
for PR 6294 at commit
[`7b1d9af`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104739810
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 thi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104739842
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 this f
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/6294#discussion_r30919012
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -82,13 +113,23 @@ class ReceiverTracker(ssc: StreamingConte
Github user harishreedharan commented on a diff in the pull request:
https://github.com/apache/spark/pull/6294#discussion_r30917969
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
---
@@ -117,11 +117,15 @@ private[streaming] abstract
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/6294#discussion_r30917648
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/receiver/ReceiverSupervisor.scala
---
@@ -117,11 +117,15 @@ private[streaming] abstract class Rece
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104450030
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104450027
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/6294#issuecomment-104449982
[Test build #33286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33286/consoleFull)
for PR 6294 at commit
[`15ed4a1`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104427221
[Test build #33286 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33286/consoleFull)
for PR 6294 at commit
[`15ed4a1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104427155
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/6294#issuecomment-104427139
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/6294#discussion_r30842963
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StreamingContextSuite.scala
---
@@ -224,6 +224,19 @@ class StreamingContextSuite extends FunSui
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/6294#discussion_r30841651
--- Diff:
streaming/src/test/scala/org/apache/spark/streaming/StreamingContextSuite.scala
---
@@ -224,6 +224,19 @@ class StreamingContextSuite extends FunSuite
Github user viirya commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104106598
Looks good.
---
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
ena
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104064806
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104064807
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104064784
[Test build #33172 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33172/consoleFull)
for PR 6294 at commit
[`fff63f9`](https://git
Github user zsxwing commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104036466
> Do you need explicit locking? Wouldn't synchronized start, register, stop
etc be enough?
Using `this` will block the `registerReceiver` method when stopping be
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104035211
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104035412
[Test build #33172 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33172/consoleFull)
for PR 6294 at commit
[`fff63f9`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104035162
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 harishreedharan commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034646
Do you need explicit locking? Wouldn't `synchronized` start, register, stop
etc be enough?
---
If your project is set up for it, you can reply to this email an
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034569
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034561
[Test build #33170 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33170/consoleFull)
for PR 6294 at commit
[`d5a1767`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034573
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034223
[Test build #33170 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33170/consoleFull)
for PR 6294 at commit
[`d5a1767`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104034038
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/6294#issuecomment-104034080
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 the pull request:
https://github.com/apache/spark/pull/6294#issuecomment-104033650
@viirya could you take a look?
cc @tdas
---
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
71 matches
Mail list logo