Skip to content

[SPARK-4141] Hide Accumulators column on stage page when no accumulators exist#3031

Closed
mmm wants to merge 4 commits into
apache:masterfrom
mmm:remove-accumulators-col
Closed

[SPARK-4141] Hide Accumulators column on stage page when no accumulators exist#3031
mmm wants to merge 4 commits into
apache:masterfrom
mmm:remove-accumulators-col

Conversation

@mmm

@mmm mmm commented Oct 31, 2014

Copy link
Copy Markdown
Contributor

WebUI

@AmplabJenkins

Copy link
Copy Markdown

Can one of the admins verify this patch?

@JoshRosen

Copy link
Copy Markdown
Contributor

Jenkins, this is ok to test.

@SparkQA

SparkQA commented Oct 31, 2014

Copy link
Copy Markdown

Test build #22604 has started for PR 3031 at commit 390893b.

  • This patch merges cleanly.

@SparkQA

SparkQA commented Oct 31, 2014

Copy link
Copy Markdown

Test build #22604 has finished for PR 3031 at commit 390893b.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@AmplabJenkins

Copy link
Copy Markdown

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

@SparkQA

SparkQA commented Oct 31, 2014

Copy link
Copy Markdown

Test build #22609 has started for PR 3031 at commit 6141cb3.

  • This patch merges cleanly.

@mmm

mmm commented Oct 31, 2014

Copy link
Copy Markdown
Contributor Author

line dragged over... sorry for the noise

@SparkQA

SparkQA commented Oct 31, 2014

Copy link
Copy Markdown

Test build #22609 has finished for PR 3031 at commit 6141cb3.

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

@AmplabJenkins

Copy link
Copy Markdown

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

@JoshRosen

Copy link
Copy Markdown
Contributor

This looks good to me. It looks like there's now a merge conflict due to me merging Kay's patch, but I'll try to fix it up myself on merge.

@JoshRosen

Copy link
Copy Markdown
Contributor

Alright, I've fixed the merge conflicts and committed this! Thanks @mmm!

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