Skip to content

Conversation

@sharkdtu
Copy link
Contributor

What changes were proposed in this pull request?

Fix code error in yarn-cluster unit test.

How was this patch tested?

Use exist tests


test("run Spark in yarn-cluster mode with different configurations") {
testBasicYarnApp(true,
testBasicYarnApp(false,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@srowen
Copy link
Member

srowen commented Jul 13, 2016

Jenkins test this please

@SparkQA
Copy link

SparkQA commented Jul 13, 2016

Test build #62232 has finished for PR 14166 at commit 816979b.

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

asfgit pushed a commit that referenced this pull request Jul 13, 2016
## What changes were proposed in this pull request?

Fix code error in yarn-cluster unit test.

## How was this patch tested?

Use exist tests

Author: sharkd <sharkd.tu@gmail.com>

Closes #14166 from sharkdtu/master.

(cherry picked from commit 3d6f679)
Signed-off-by: Sean Owen <sowen@cloudera.com>
@srowen
Copy link
Member

srowen commented Jul 13, 2016

Merged to master/2.0

@asfgit asfgit closed this in 3d6f679 Jul 13, 2016
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