Skip to content
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

[SPARK-5401] set executor ID before creating MetricsSystem #4194

Closed
wants to merge 1 commit into from
Closed

[SPARK-5401] set executor ID before creating MetricsSystem #4194

wants to merge 1 commit into from

Conversation

ryan-williams
Copy link
Contributor

No description provided.

@SparkQA
Copy link

SparkQA commented Jan 25, 2015

Test build #26063 has started for PR 4194 at commit 7c5a33f.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Jan 25, 2015

Test build #26063 has finished for PR 4194 at commit 7c5a33f.

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

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/26063/
Test PASSed.

@andrewor14
Copy link
Contributor

Good catch. LGTM and I will merge this into master thanks

@asfgit asfgit closed this in 2d9887b Jan 25, 2015
asfgit pushed a commit that referenced this pull request Jan 25, 2015
@ryan-williams
[SPARK-5401] set executor ID before creating MetricsSystem
1f8b718 
Author: Ryan Williams <ryan.blake.williams@gmail.com>

Closes #4194 from ryan-williams/metrics and squashes the following commits:

7c5a33f [Ryan Williams] set executor ID before creating MetricsSystem
ryan-williams added a commit to ryan-williams/spark that referenced this pull request Jan 27, 2015
@ryan-williams
Remove redundant executor-id set() call
b3e4f7b 
This happens inside SparkEnv initialization as of apache#4194
@ryan-williams ryan-williams mentioned this pull request Jan 27, 2015
Closed
asfgit pushed a commit that referenced this pull request Jan 28, 2015
@ryan-williams
[SPARK-5417] Remove redundant executor-id set() call
a731314 
This happens inside SparkEnv initialization as of #4194

Author: Ryan Williams <ryan.blake.williams@gmail.com>

Closes #4213 from ryan-williams/exec-id-set and squashes the following commits:

b3e4f7b [Ryan Williams] Remove redundant executor-id set() call
asfgit pushed a commit that referenced this pull request Feb 27, 2015
@ryan-williams @srowen
[SPARK-5417] Remove redundant executor-id set() call
17b7cc7 
This happens inside SparkEnv initialization as of #4194

Author: Ryan Williams <ryan.blake.williams@gmail.com>

Closes #4213 from ryan-williams/exec-id-set and squashes the following commits:

b3e4f7b [Ryan Williams] Remove redundant executor-id set() call
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.
4 participants
@ryan-williams @SparkQA @AmplabJenkins @andrewor14