Skip to content

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