Skip to content

[SPARK-25550][WEBUI]Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobs - #22567

Closed
shivusondur wants to merge 3 commits into
apache:masterfrom
shivusondur:EnvPageHistory
Closed

[SPARK-25550][WEBUI]Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobs#22567
shivusondur wants to merge 3 commits into
apache:masterfrom
shivusondur:EnvPageHistory

Conversation

@shivusondur

@shivusondurshivusondur commented Sep 27, 2018

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Currently, in History spark UI, the number of Jobs in the jobs page and "spark.ui.retainedJobs" value in
the Environment page is not matching.
So we need to override the spark properties from the History spark-default.conf properties, which will match the number of jobs showing in jobs page and value of "spark.ui.retainedJobs" in the Environment Page.
Same applies to other History specific parameters such as "spark.ui.retainedTasks" etc.

How was this patch tested?

Tested in master branch

  1. In the spark-default.conf mentioned the spark.ui.retainedJobs=200
  2. In spark shell command mentioned the 100 like below command
    "spark-shell --master yarn --conf spark.ui.retainedJobs=100"

Attached the test result screenshot
spark.ui.retainedJobs is showing specific to historypage
image

Now the total jobs showing and the spark.ui,retainedJobs value in the Environment page are matching.

46128573-87b92f00-c251-11e8-926c-030d86e15ad5

@shivusondurshivusondur changed the title [SPARK-25550] [Spark Job History] Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobs[SPARK-25550] [WEBUI] Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobsSep 27, 2018
@shivusondurshivusondur changed the title [SPARK-25550] [WEBUI] Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobs[WIP]Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobsSep 27, 2018
@shivusondurshivusondur changed the title [WIP]Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobs[SPARK-25550][WEBUI]Environment Page of Spark Job History UI showing wrong value for spark.ui.retainedJobsSep 27, 2018
@shivusondur

Copy link
Copy Markdown
ContributorAuthor

@vanzin Please review this patch.

@vanzin

Copy link
Copy Markdown
Contributor

Hmmm, not sure about this. That page is supposed to show the application's configuration, not the history server configuration.

If you want to add a new page to show SHS-specific configs, then that would be a better change. Otherwise this can be misleading. e.g. knowing the original config value can help someone debug things if the app ran into OOM issues.

Otherwise I don't think this should go in.

@shivusondur

shivusondur commented Oct 9, 2018

Copy link
Copy Markdown
ContributorAuthor

@vanzin
I modified according to your suggestion and updated the code.
Now History properties will come as seperate tab in the EnvironmentPage as shown below snap.

History UI
screenshot from 2018-10-09 11-47-07

screenshot from 2018-10-09 12-03-12

Only in the history UI, History Properties will be displayed. In live UI history properties won't be there

Live UI
screenshot from 2018-10-09 12-00-11

@vanzin

Copy link
Copy Markdown
Contributor

That has the same problem as before. This page is for the application, not for the history server. It should not show different things just because you're looking at it in the history server.

If you want to expose the SHS config you have to do it in a separate page that is not part of the application-specific UI. And then you have to be careful with what you show there, since the SHS config may have sensitive information that users should not see.

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@srowensrowen mentioned this pull request Oct 24, 2018
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 22, 2025
Closesapache#22567Closesapache#18457Closesapache#21517Closesapache#21858Closesapache#22383Closesapache#19219Closesapache#22401Closesapache#22811Closesapache#20405Closesapache#21933Closesapache#22819 from srowen/ClosePRs.
Authored-by: Sean Owen <sean.owen@databricks.com>
Signed-off-by: Sean Owen <sean.owen@databricks.com>
Sign up for freeto 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.

3 participants

@shivusondur@vanzin@AmplabJenkins