Skip to content

SPARK-1291: Link the spark UI to RM ui in yarn-client mode #1112

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 8 commits into from

Conversation

witgo
Copy link
Contributor

@witgo witgo commented Jun 18, 2014

No description provided.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/15869/

logInfo("Registering the ApplicationMaster")
// TODO: Find out client's Spark UI address and fill in here?
amClient.registerApplicationMaster(Utils.localHostName(), 0, "")
val appUIAddress = sparkConf.getOption("spark.driver.appUIAddress").getOrElse("")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can just use sparkConf.get("spark.driver.appUIAddress", "")

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16030/

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished.

@AmplabJenkins
Copy link

Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16047/

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished.

@AmplabJenkins
Copy link

Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16099/

@witgo
Copy link
Contributor Author

witgo commented Jun 25, 2014

Jenkins, test this please.

@AmplabJenkins
Copy link

Merged build triggered.

@AmplabJenkins
Copy link

Merged build started.

@AmplabJenkins
Copy link

Merged build finished. All automated tests passed.

@AmplabJenkins
Copy link

All automated tests passed.
Refer to this link for build results: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/16100/

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

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

@witgo
Copy link
Contributor Author

witgo commented Jul 15, 2014

@tgravescs The code has been submitted. Because I don't have the hadoop 0.23.x cluster, the code no strict test.

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

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

@tgravescs
Copy link
Contributor

Thanks, I'll test out the 0.23 code.

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

QA results for PR 1112:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds the following public classes (experimental):
case class AddWebUIFilter(filterName:String, filterParams: String, proxyBase :String)

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

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

QA results for PR 1112:
- This patch FAILED unit tests.
- This patch merges cleanly
- This patch adds the following public classes (experimental):
case class AddWebUIFilter(filterName:String, filterParams: String, proxyBase :String)

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

@tgravescs
Copy link
Contributor

Jenkins, test this please

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

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

@SparkQA
Copy link

SparkQA commented Jul 15, 2014

QA results for PR 1112:
- This patch PASSES unit tests.
- This patch merges cleanly
- This patch adds the following public classes (experimental):
case class AddWebUIFilter(filterName:String, filterParams: String, proxyBase :String)

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

@tgravescs
Copy link
Contributor

Looks good. Thanks @witgo

@asfgit asfgit closed this in 72ea56d Jul 15, 2014
gzm55 pushed a commit to MediaV/spark that referenced this pull request Jul 18, 2014
Author: witgo <[email protected]>

Closes apache#1112 from witgo/SPARK-1291 and squashes the following commits:

6022bcd [witgo] review commit
1fbb925 [witgo] add addAmIpFilter to yarn alpha
210299c [witgo] review commit
1b92a07 [witgo] review commit
6896586 [witgo] Add comments to addWebUIFilter
3e9630b [witgo] review commit
142ee29 [witgo] review commit
1fe7710 [witgo] Link the spark UI to RM ui in yarn-client mode

Conflicts:
	yarn/stable/src/main/scala/org/apache/spark/deploy/yarn/ExecutorLauncher.scala
asfgit pushed a commit that referenced this pull request Jul 23, 2014
The issue is caused by #1112 .

Author: GuoQiang Li <[email protected]>

Closes #1501 from witgo/webui_style and squashes the following commits:

4b34998 [GuoQiang Li] In some cases, pages display incorrect in WebUI
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
Author: witgo <[email protected]>

Closes apache#1112 from witgo/SPARK-1291 and squashes the following commits:

6022bcd [witgo] review commit
1fbb925 [witgo] add addAmIpFilter to yarn alpha
210299c [witgo] review commit
1b92a07 [witgo] review commit
6896586 [witgo] Add comments to addWebUIFilter
3e9630b [witgo] review commit
142ee29 [witgo] review commit
1fe7710 [witgo] Link the spark UI to RM ui in yarn-client mode
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
The issue is caused by apache#1112 .

Author: GuoQiang Li <[email protected]>

Closes apache#1501 from witgo/webui_style and squashes the following commits:

4b34998 [GuoQiang Li] In some cases, pages display incorrect in WebUI
mapr-devops pushed a commit to mapr/spark that referenced this pull request May 8, 2025
…verride option (apache#1112)

[EZSPA-1073] Add ssibility to work with spark.rapids.shims-provider-override option

---------

Co-authored-by: Egor Krivokon <>
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