Skip to content

[SPARK-5634] [core] Show correct message in HS when no incomplete apps f... - #5515

Closed
vanzin wants to merge 1 commit into
apache:masterfrom
vanzin:SPARK-5634
Closed

[SPARK-5634] [core] Show correct message in HS when no incomplete apps f...#5515
vanzin wants to merge 1 commit into
apache:masterfrom
vanzin:SPARK-5634

Conversation

@vanzin

@vanzinvanzin commented Apr 14, 2015

Copy link
Copy Markdown
Contributor

...ound.

@SparkQA

SparkQA commented Apr 15, 2015

Copy link
Copy Markdown

Test build #30283 has finished for PR 5515 at commit f74ecf1.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@andrewor14

andrewor14 commented Apr 15, 2015

Copy link
Copy Markdown
Contributor

LGTM merging into master 1.3 and 1.2 thanks.

asfgit pushed a commit that referenced this pull request Apr 15, 2015
Marcelo VanzinAndrew Or
[SPARK-5634] [core] Show correct message in HS when no incomplete app…
0e5ca9e
…s f...
...ound.
Author: Marcelo Vanzin <vanzin@cloudera.com>
Closes#5515 from vanzin/SPARK-5634 and squashes the following commits:
f74ecf1 [Marcelo Vanzin] [SPARK-5634] [core] Show correct message in HS when no incomplete apps found.
(cherry picked from commit 30a6e0d)
Signed-off-by: Andrew Or <andrew@databricks.com>
asfgit pushed a commit that referenced this pull request Apr 15, 2015
Marcelo VanzinAndrew Or
[SPARK-5634] [core] Show correct message in HS when no incomplete app…
5845a62
…s f...
...ound.
Author: Marcelo Vanzin <vanzin@cloudera.com>
Closes#5515 from vanzin/SPARK-5634 and squashes the following commits:
f74ecf1 [Marcelo Vanzin] [SPARK-5634] [core] Show correct message in HS when no incomplete apps found.
(cherry picked from commit 30a6e0d)
Signed-off-by: Andrew Or <andrew@databricks.com>
@JoshRosen

JoshRosen commented Apr 15, 2015

Copy link
Copy Markdown
Contributor

I'm going to revert this patch in branch-1.2 (1.2.3) because it broke compilation there (the requestedIncomplete isn't defined in that branch). If we want this fix for 1.2, we can open a new PR, but this seems like such a low-impact problem that it's probably not worth fixing there.

@andrewor14

andrewor14 commented Apr 15, 2015

Copy link
Copy Markdown
Contributor

Oops, my mistake. Thanks Josh.

@vanzin
vanzin deleted the SPARK-5634 branch April 16, 2015 16:31
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vanzin@SparkQA@andrewor14@JoshRosen