Skip to content

[SPARK-5736][Web UI]Add executor log url to Executors page on Yarn #4529

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.

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

Closed
wants to merge 1 commit into from
Closed

[SPARK-5736][Web UI]Add executor log url to Executors page on Yarn #4529

wants to merge 1 commit into from

Conversation

shenh062326
Copy link
Contributor

Currently, there is not executor log url in spark ui (on Yarn), we have to read executor log by login the machine that executor in. I think we should add executor log url to executors pages.

@SparkQA
Copy link

SparkQA commented Feb 11, 2015

Test build #27285 has started for PR 4529 at commit 078ca72.

  • This patch does not merge cleanly.

@srowen
Copy link
Member

srowen commented Feb 11, 2015

This needs a rebase already. Is it really necessary to push through UI info like this into all the core code?

@shenh062326
Copy link
Contributor Author

Hi @srowen.
We just want to read executor log from UI. is there any easy way to add executor log url to UI?

@SparkQA
Copy link

SparkQA commented Feb 11, 2015

Test build #27285 has finished for PR 4529 at commit 078ca72.

  • This patch fails Spark unit tests.
  • This patch does not merge cleanly.
  • This patch adds the following public classes (experimental):
    • case class SparkListenerBlockManagerAdded(
    • class StorageStatus(

@AmplabJenkins
Copy link

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

@srowen
Copy link
Member

srowen commented Feb 11, 2015

Sandy's right, I forgot this was just committed last week. https://issues.apache.org/jira/browse/SPARK-2450

@andrewor14
Copy link
Contributor

Hey @shenh062326 there's already a solution for this that was merged in #3486. Would you mind closing this PR?

@asfgit asfgit closed this in 24f358b Feb 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants