Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-50796342
Can you please add unit test for this change.
---
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 user tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-50933873
Also could you change the title to contain "SPARK-2381" (withi -)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-54679164
Jenkins, this is ok to test.
---
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 tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-54679375
@joyyoj Can you please add an unit test for this behavior in the
StremaingContextSuite, this is a significant change in the program behavior and
should have a unit test.
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/1693#discussion_r17197213
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -272,7 +272,15 @@ class ReceiverTracker(ssc: StreamingConte
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/1693#discussion_r17197419
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -272,7 +272,15 @@ class ReceiverTracker(ssc: StreamingConte
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-54694532
Can one of the admins verify this patch?
---
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 d
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-56858048
@joyyoj any updates on this?
---
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 joyyoj commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-51146846
Ok, I'll do it soon
---
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 feat
Github user tdas commented on the pull request:
https://github.com/apache/spark/pull/1693#issuecomment-6999
@joyyoj If you are unable to address the comments on this PR, mind closing
this?
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/1693#discussion_r20042431
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/ReceiverTracker.scala
---
@@ -272,7 +272,15 @@ class ReceiverTracker(ssc: StreamingCon
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/1693
---
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
12 matches
Mail list logo