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.

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

[SPARK-3611] [Web UI] Show number of cores for each executor in application web UI #11039

Closed
wants to merge 1 commit into from

Conversation

ajbozarth
Copy link
Member

Added a Cores column in the Executors UI

@ajbozarth
Copy link
Member Author

Since we have api access to the information now as I discovered in #10154 I've added the Cores column.

Do we want to move where it is located in the table?

spark-3611_cores
[Note: this screenshot is take with the fix in #11038]

@ajbozarth
Copy link
Member Author

looping in @tgravescs @zsxwing @JoshRosen since you reviewed the first PR

@SparkQA
Copy link

SparkQA commented Feb 3, 2016

Test build #50628 has finished for PR 11039 at commit ea8538c.

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

@holdenk
Copy link
Contributor

holdenk commented Feb 3, 2016

Seems like the test failure is unrelated (I got the same in one of my PRs too). Jenkins retest this please.

@ajbozarth
Copy link
Member Author

Can't seem to win with Jenkins. Jenkins retest this please.

@zsxwing
Copy link
Member

zsxwing commented Feb 3, 2016

retest this please

@SparkQA
Copy link

SparkQA commented Feb 3, 2016

Test build #50681 has finished for PR 11039 at commit ea8538c.

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

@zsxwing
Copy link
Member

zsxwing commented Feb 3, 2016

LGTM. Merging to master. Thanks, @ajbozarth

@asfgit asfgit closed this in 3221edd Feb 3, 2016
@ajbozarth ajbozarth deleted the spark3611 branch February 4, 2016 01:23
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.

4 participants