Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169631635
OK, I see you replaced a lot more with interpolation. I guess that's OK.
But I think it's why you have a merge conflict now that needs to be resolved.
---
If your
Github user jaceklaskowski commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169633803
On it. I'll rebase and push update. Thanks @srowen for code review.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169686335
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169686651
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169690276
**[Test build #48944 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48944/consoleFull)**
for PR 10595 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169729078
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10595
---
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
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-169806744
Merged to master
---
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
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168933998
Can you combine your pull requests into a single one?
---
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 jaceklaskowski opened a pull request:
https://github.com/apache/spark/pull/10595
[STREAMING][MINOR] More contextual information in logs + minor code iâ¦
â¦mprovements
Please review and merge at your convenience. Thanks!
You can merge this pull request into a
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-16890
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
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48830059
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala ---
@@ -59,17 +59,15 @@ case class JobSet(
// Time
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48828208
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/DStream.scala ---
@@ -286,7 +286,7 @@ abstract class DStream[T: ClassTag] (
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48828188
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala ---
@@ -59,17 +59,15 @@ case class JobSet(
// Time taken to
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48828135
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala ---
@@ -59,17 +59,15 @@ case class JobSet(
// Time taken to
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48830663
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/scheduler/JobSet.scala ---
@@ -59,17 +59,15 @@ case class JobSet(
// Time
Github user jaceklaskowski commented on a diff in the pull request:
https://github.com/apache/spark/pull/10595#discussion_r48830750
--- Diff:
streaming/src/main/scala/org/apache/spark/streaming/dstream/DStream.scala ---
@@ -286,7 +286,7 @@ abstract class DStream[T: ClassTag] (
Github user jaceklaskowski commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168949749
Yes, sure! Been worried that @srowen might cross few lines out that would
completely devastate my mood today :) On to...
---
If your project is set up for it,
Github user jaceklaskowski commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168975843
Merged the other branches and ran build locally. Please review and merge at
your convenience @srowen @rxin. Thanks!
---
If your project is set up for it, you
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168978428
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168979320
**[Test build #48761 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48761/consoleFull)**
for PR 10595 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168997214
**[Test build #48761 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/48761/consoleFull)**
for PR 10595 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168997399
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10595#issuecomment-168997402
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
24 matches
Mail list logo
