Skip to content

SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetrics... #1961

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
sryza wants to merge 2 commits into from
Closed

SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetrics... #1961

sryza wants to merge 2 commits into from

Conversation

@sryza
Copy link
Contributor

@sryza sryza commented Aug 15, 2014

...Update

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1961. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18591/consoleFull

@rxin
Copy link
Contributor

rxin commented Aug 15, 2014

LGTM pending Jenkins.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA results for PR 1961:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds no public classes

For more information see test ouptut:
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18591/consoleFull

@rxin
Copy link
Contributor

rxin commented Aug 15, 2014

Oops compilation failed.

@sryza
Copy link
Contributor Author

sryza commented Aug 15, 2014

Ooops, fixed.

@andrewor14
Copy link
Contributor

andrewor14 commented Aug 15, 2014

LGTM

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1961. This patch merges cleanly.
View progress: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/18595/consoleFull

@pwendell
Copy link
Contributor

pwendell commented Aug 15, 2014

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1961 at commit dccdff5.

  • This patch merges cleanly.

@pwendell
Copy link
Contributor

pwendell commented Aug 15, 2014

Jenkins, retest this please.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have started for PR 1961 at commit dccdff5.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 15, 2014

QA tests have finished for PR 1961 at commit dccdff5.

  • This patch passes unit tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@pwendell
Copy link
Contributor

pwendell commented Aug 15, 2014

Okay looks good - I'm merging this.

asfgit pushed a commit that referenced this pull request Aug 15, 2014
@sryza @pwendell
SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetr…
63376a0 
…ics...

...Update

Author: Sandy Ryza <[email protected]>

Closes #1961 from sryza/sandy-spark-3028 and squashes the following commits:

dccdff5 [Sandy Ryza] Fix compile error
f883ded [Sandy Ryza] SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetricsUpdate
(cherry picked from commit 0afe5cb)

Signed-off-by: Patrick Wendell <[email protected]>
@asfgit asfgit closed this in 0afe5cb Aug 15, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
@sryza @conviva-zz
SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetr…
4ba1686 
…ics...

...Update

Author: Sandy Ryza <[email protected]>

Closes apache#1961 from sryza/sandy-spark-3028 and squashes the following commits:

dccdff5 [Sandy Ryza] Fix compile error
f883ded [Sandy Ryza] SPARK-3028. sparkEventToJson should support SparkListenerExecutorMetricsUpdate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@sryza @SparkQA @rxin @andrewor14 @pwendell