Github user jerryshao closed the pull request at:
https://github.com/apache/spark/pull/9238
---
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 e
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-164621907
OK, I see. I will close this PR, thanks a lot for your comments.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-164621364
@jerryshao would you mind closing this PR and maybe re-opening one that
only supports the download functionality?
---
If your project is set up for it, you can reply
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-164621244
The download functionality is super cool, but the import part has somewhat
confusing semantics. The history server is supposed to dynamically sync all
logs in a speci
Github user steveloughran commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151462264
1. Adding new abstract methods is going to break providers which use the
current plugin point. Adding base implementations which throw an "unsupported"
exception w
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/9238#discussion_r43109175
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/FsHistoryProvider.scala ---
@@ -299,6 +300,54 @@ private[history] class FsHistoryProvider(
Github user steveloughran commented on a diff in the pull request:
https://github.com/apache/spark/pull/9238#discussion_r43100632
--- Diff:
core/src/main/scala/org/apache/spark/deploy/history/ApplicationHistoryProvider.scala
---
@@ -73,4 +73,11 @@ private[history] abstract class
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151029368
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/9238#issuecomment-151029319
**[Test build #44335 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44335/consoleFull)**
for PR 9238 at commit
[`af8b3cb`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151029370
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151014490
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/9238#issuecomment-151014493
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/9238#issuecomment-151014535
**[Test build #44335 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44335/consoleFull)**
for PR 9238 at commit
[`af8b3cb`](https://gith
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151014208
Jenkins, retest 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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151009668
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-151009667
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/9238#issuecomment-151009628
**[Test build #44324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44324/consoleFull)**
for PR 9238 at commit
[`af8b3cb`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150995951
**[Test build #44324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44324/consoleFull)**
for PR 9238 at commit
[`af8b3cb`](https://gith
Github user jerryshao commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150995617
Updated UI:
---
If your project is set up for
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150484503
**[Test build #44197 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44197/consoleFull)**
for PR 9238 at commit
[`f92393f`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150484655
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150484654
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/9238#issuecomment-150467073
**[Test build #44197 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44197/consoleFull)**
for PR 9238 at commit
[`f92393f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/9238#issuecomment-150464255
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/9238#issuecomment-150464301
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
25 matches
Mail list logo
